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.

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

HMS-800-2T #2152

Closed
3 tasks done
Vicio88 opened this issue Jul 17, 2024 · 15 comments
Closed
3 tasks done

HMS-800-2T #2152

Vicio88 opened this issue Jul 17, 2024 · 15 comments
Labels
bug Something isn't working

Comments

@Vicio88
Copy link

Vicio88 commented Jul 17, 2024

What happened?

Buon giorno. Ho un problema con la mia dtu. Non riconosce il mio inverter HMS-800-2T. Ho fatto anche l’ultimo aggiornamento ma non va

To Reproduce Bug

Buon giorno. Ho un problema con la mia dtu. Non riconosce il mio inverter HMS-800-2T. Ho fatto anche l’ultimo aggiornamento ma non va

Expected Behavior

v4.4.7-dirtyI

Install Method

Pre-Compiled binary from GitHub

What git-hash/version of OpenDTU?

v4.4.7-dirty

Relevant log/trace output

No response

Anything else?

No response

Please confirm the following

  • I believe this issue is a bug that affects all users of OpenDTU, not something specific to my installation.
  • I have already searched for relevant existing issues and discussions before opening this report.
  • I have updated the title field above with a concise description.
@Vicio88 Vicio88 added the bug Something isn't working label Jul 17, 2024
@broth-itk
Copy link
Contributor

Please provide more information like S/N of your inverter and the correct OpenDTU version.
Screenshot would be nice.
v4.4.7-dirtyI is no version from an OpenDTU release

@Vicio88
Copy link
Author

Vicio88 commented Jul 17, 2024

IMG_9970
IMG_9971
IMG_9972

@Vicio88
Copy link
Author

Vicio88 commented Jul 17, 2024

Inverte hoymiles HMS-800-2T

@stefan123t
Copy link
Contributor

stefan123t commented Jul 17, 2024

@Vicio88 we need at least the first seven digits of your inverter serial id.
The last five are unique and can be ommitted.

Do you have two different serial IDs on the sticker, ie the model number ( first four digits ) may differ for HMS-800W-2T Models as they have a built-in DTUBI as well.

@Vicio88
Copy link
Author

Vicio88 commented Jul 17, 2024

IMG_9973

@stefan123t
Copy link
Contributor

Your first inverter is a HMS-800W model. The second most likely too, given the same model prefix.
You can use any of the other projects for DTUBI and HMS-xxxW HMS-WiFi-Series Models or directly use the S-Miles Installer App / S-Miles Cloud from Hoymiles itself.

@Vicio88
Copy link
Author

Vicio88 commented Jul 17, 2024

Dove lo trovo ?

@stefan123t
Copy link
Contributor

Ciao Vicio, the most known is this one written in RUST from DennisOSRM https://github.com/DennisOSRM/hms-mqtt-publisher
At the moment it does support only a single HMS-WiFi-Series, though there is a pull request # 94 to support multiple inverters too.

I do prefer the Python implementation by rovo89 as it is more promising, but I do not own an HMS-WiFi-Series inverter so this is only my personal point of view.

@broth-itk
Copy link
Contributor

Maybe it's good idea to add all known serial number prefixes to the list and name them "Unsupported" or "WIFI Unsupported"

@tbnobody
Copy link
Owner

tbnobody commented Aug 5, 2024

I added a open topic to the documentation project and will close this issue.

@tbnobody tbnobody closed this as not planned Won't fix, can't repro, duplicate, stale Aug 5, 2024
@stefan123t
Copy link
Contributor

@tbnobody so we can not identify them based on the Serial ID users enter ?
This would allow us to show an alert / warning in the add/edit inverter dialog to notify users

I answer several reports / discussions per week and have to explain the distinction between vanilla HMS and HMS-xxxW again and again.

Maybe it would be good to hint at this fact also in the Bug template or pose it as a checkbox / question ?

Kind regards, Stefan

@tbnobody
Copy link
Owner

tbnobody commented Aug 5, 2024

I will add a checkbox to the issue template. I'm not really willing to add a blacklist to the WebGUI, especially as an "Unknown" etc. is also displayed in the inverter overview. It will be also mentioned in the documentation. I mean, in how many other places can people ignore it?

@stefan123t
Copy link
Contributor

Well if they can ignore it, they will. 🤣
Even if a clear Warning its there they will try to ignore it if possible.

One last thought, though you are reluctant to add support for those built-in DTUBI HMS WiFi series inverters, they are not Unknown but just Unsupported at the moment.

Maybe we should add a stub for a protobuf implementation via a separate AP network link.
Once the Setup AP has been closed we could use its connection to connect to at least one of the DTUBI APs.
I wonder if we would have to connect to each AP in case we want to query multiple such DTUBI based inverters ?

@broth-itk
Copy link
Contributor

IMHO I agree to stefan123t, there should be a "Unsupported" message in the GUI when adding a W inverter.
Since asking questions is always easier as reading documentaton or even the text beside of checkboxes, this change will help keeping motivation up for all involved in OpenDTU development and support (via Issues).

Just my 2 cents

Copy link

github-actions bot commented Sep 6, 2024

This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new discussion or issue for related concerns.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 6, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

4 participants