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

Deactivate/ hide battery features #65

Closed
i-peter opened this issue Feb 16, 2023 · 3 comments
Closed

Deactivate/ hide battery features #65

i-peter opened this issue Feb 16, 2023 · 3 comments

Comments

@i-peter
Copy link

i-peter commented Feb 16, 2023

Hi,
first of all: Thanks for your great work!

I have a SH8.0RT and it's working great for me. Althought it was hard to find out, that I have to use the second LAN port next to the WiNet-S dongle to make it work. (BTW: Is there anywhere a guideline how to upgrade firmware for Sungrow? I only found outdated descriptions, which are not working for me.)

The question/ feature request I really have is the following:
I don't have a battery and I would like to deactivate or hide all battery functionallities.

Would be great to have a central parameter instead of going through all lines of code to deactivate.

Best regards,
Iven

@mkaiser
Copy link
Owner

mkaiser commented Feb 16, 2023

hi

I have a SH8.0RT and it's working great for me. Althought it was hard to find out, that I have to use the second LAN port next to the WiNet-S dongle to make it work.

I have the first revision of the SH10.RT without WiNet-S, so I can't really say something about all this. I just know, that putting the cable in the WiNetS port often causes problems :/

Now, that you figured it out, could you give me some hints to update the documentation for this? I am pretty sure that would help others, too :)

I don't have a battery and I would like to deactivate or hide all battery functionallities.
hmm the only way I see at the moment is to split the modbus_sungrow.yaml in multiple files, lets say "basic", "battery" and "history".
I am afraid, that a more dynamic way is not possible in the yaml (there is no yaml-wide if/else possible).

Will give the splitting some thoughts in the future :)

With a more versatile Python integration for Home Assistant this would be much easier to achieve and more dynamic (just include feature XY, if checkbox is marked). Unfortunately, I have never used Python, yet and quiet frankly I don't have the time at the moment to learn it.

If someone knows how to do this, I would try to help :)

@mkaiser
Copy link
Owner

mkaiser commented Feb 28, 2023

I am stuck trying to separate the config file. It would be nice if you can just put a .yaml file in a modbus_sungrow folder. If the yaml exists, it will be loaded. My tests ended up in crashes :/

@mkaiser
Copy link
Owner

mkaiser commented Sep 11, 2023

had many thoughts about this.

I only have the capacity to deal with one integration variant (so the one with battery).

If you do not want them, delete the YAML entries.

I am sorry, but there is no other way with a pure YAML-integration. Only python could fix this

@mkaiser mkaiser closed this as completed Sep 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants