Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Adding some guidance on how to get the huawei-sun2000-dongle-powersensor configured correctly #13989

Merged
merged 3 commits into from
May 29, 2024
Merged

Adding some guidance on how to get the huawei-sun2000-dongle-powersensor configured correctly #13989

merged 3 commits into from
May 29, 2024

Conversation

natsu-chan
Copy link
Contributor

adding information about how to activate modus/tcp within the inverter, since the default does not allow to connect via modes/tcp.

@natsu-chan
Adding some guidance on how to get the huawei-sun2000-dongle-powersen…
02dda8f 
…sor configured correctly

adding information about how to activate modus/tcp within the inverter, since the default does not allow to connect via modes/tcp.
@andig andig added the devices Specific device support label May 21, 2024
andig
andig reviewed May 21, 2024
View reviewed changes
templates/definition/meter/huawei-sun2000-dongle-powersensor.yaml Outdated
requirements:
description:
de: |
Es kann notwendig sein, dass der Installateur mit Hilfe eines "Errichterzugangs" zunächst "Modbus/TCP" in den Kommunikationseinstellungen des Wechselrichters freischaltet.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ich fände eine knappe Formulierung besser:

Erfordert ModbusTCP. Siehe ...

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ich bin auch immer für knappe Formulierungen, aber ich hätte mit "Erfordert MobusTCP" nichts anfangen können :D Dann wäre ich nicht in der Lage gewesen meinem Installateur zu sagen, was ich haben will....
Und der Link könnte potentiell in Zukunft mal kaputt gehen, wenn Huawei die Doku umzieht...

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more. 

Erfordert ModbusTCP. Freischaltung über den "Errichterzugang". Siehe ...

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

passe ich an :)

@andig
Copy link
Member

andig commented May 24, 2024

Do we need the same comment in the non-powersensor tempalte?

@andig andig requested a review from premultiply May 24, 2024 10:07
@natsu-chan
Copy link
Contributor Author

Do we need the same comment in the non-powersensor tempalte?

I'm not sure. Might be the case.
I don't know whether this configuration takes place in the dongle or in the inverter itself.
Maybe one could add it with a "might be the case" in front of it? And hope somebody in the future gives feedback whether it is really necessary?

@andig
Copy link
Member

andig commented May 24, 2024

Dongle is used in both tempaltes!

@andig andig marked this pull request as draft May 24, 2024 15:51
@premultiply
Copy link
Member

/cc @RTTTC

@RTTTC
Copy link
Contributor

RTTTC commented May 25, 2024

Do we need the same comment in the non-powersensor tempalte?

Yes we do.

@RTTTC
Copy link
Contributor

RTTTC commented May 25, 2024

While we are fixing these templates - one template has:
energy decode: uint32nan and the other one has decode: uint32
power int32nan and int32
Which one is right / better (nan vs no nan) ( and why?)

PS - do we use energy in PV meter at all ?? I thought - only power

@natsu-chan
Copy link
Contributor Author

Do we need the same comment in the non-powersensor tempalte?

Yes we do.

I will add that.

@natsu-chan
Copy link
Contributor Author

While we are fixing these templates - one template has: energy decode: uint32nan and the other one has decode: uint32 power int32nan and int32 Which one is right / better (nan vs no nan) ( and why?)

PS - do we use energy in PV meter at all ?? I thought - only power

I have no idea, could you tackle that topic seperately? Actually has nothing todo with my topic.

@natsu-chan
Adding some guidance on how to get the huawei-sun2000-dongle configur…
4bc0420 
…ed correctly

adding information about how to activate modus/tcp within the inverter, since the default does not allow to connect via modes/tcp.
@andig andig marked this pull request as ready for review May 29, 2024 16:23
@andig andig merged commit ddfc2fb into evcc-io:master May 29, 2024
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@andig andig andig left review comments

@premultiply premultiply Awaiting requested review from premultiply

Assignees
No one assigned
Labels
devices Specific device support
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@natsu-chan @andig @premultiply @RTTTC