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

OpenDTU cannot read from HMS-inverter through CMT2300A when "Send commands" is off #1127

Closed
IdrinkOban opened this issue Jul 8, 2023 · 2 comments
Closed

OpenDTU cannot read from HMS-inverter through CMT2300A when "Send commands" is off #1127

IdrinkOban opened this issue Jul 8, 2023 · 2 comments
Labels
bug Something isn't working

Comments

@IdrinkOban
Copy link

What happened?

Depending on the configuration of an HMS-inverter, OpenDTU cannot read data from that inverter.

To Reproduce Bug

Disable the option "Send commands" in "Inverter Settings" for an HMS-inverter.
After this, OpenDTU cannot connect to this inverter any longer.
This looks like a timing problem to me.

Expected Behavior

The option "Send commands" should not affect the ability of OpenDTU, to read data from an HMS-inverter.
BZW: this is working for HM-inverter via nRF24.

Install Method

Self-Compiled

What git-hash/version of OpenDTU?

Tag v23.6.28

Relevant log/trace output

Workaroung:
Just keep the option "Send commands" turned off, even if you don't want to send commands.

Anything else?

Thank you for OpenDTU! It is awsome! :
@IdrinkOban IdrinkOban added the bug Something isn't working label Jul 8, 2023
@IdrinkOban
Copy link
Author

@tbnobody I've tested this fix during the last weekend. It is working great! Thanks a lot for providing this fix so quickly! 🙏

LennartF22 pushed a commit to LennartF22/OpenDTU that referenced this issue Aug 2, 2023
@tbnobody @LennartF22
Fix: Allow sending of ChannelChangeCommand even if EnableCommands is …
678ef58 
…disabled

This is required to successfull move the inverter to another frequency and then polling it. Without this command its not even possible to poll a inverter.

Fixes tbnobody#1127
@github-actions GitHub Actions
Copy link

github-actions bot commented Apr 4, 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 Apr 4, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@IdrinkOban