-
Notifications
You must be signed in to change notification settings - Fork 95
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
entity not available: sensor.sg_battery_charge_nom and sg_battery_level_nom #88
Comments
sorry, I missed this issue. Will take care of it, soon! :) Thank you for reporting! |
@ALL: Can you give me some feedback, if this fix is working and if the instructions are "human understandable"? ;) |
@mkaiser - thx a lot for this fix but also for this really great integration. I am a proud owner of a solar plant since one week. :) FeedbackInstructions: yes, they are easily understandable Results in my case Besides some Powerwalker sensors containing "nominal" in their name I only had two of your sensors listed in DevTools: I deleted those and after restarting HA, "Battery charge (nominal)" is "unavailable" I've checked in before and after the update -> no change
-> I've still an entity called "sensor.battery_charge_nominal" in the registry. In your .yaml file such an entity doesn't exist anymore. Any idea how to fix this? FYI: I prepared this integration in May and updated it last week on Monday (copy/paste modbus_sungrow.yaml and dashboard.yaml). |
hi, thanks for the feedback! :)
last Friday I made the fixes, so after you upgraded: Can you upgrade to the newest git-revisions?
This seems to be correct:
HA will use "battery_charge_nominal" as internal name |
Sry, of course I did upgrade before implementing the change :) I've just repeated all steps again and now both sensors are working. Don't know why exactly but I assume the first copy/paste from github to my modbus_sungrow.yaml failed somehow. I found some differences in modbus_sungrow.yaml before trying a second time. So sorry for the inconvenience! -> I now can confirm that your fix works like a charm! PS: I have a Sungrow SBR128 battery and sensor.battery_capacity always shows 0. But this is another topic and I don't want to hijack this thread ;) |
I am allowed to hijack ;) |
Hello Very good stuff. I am using this a lot. But i wounder if it is also possible to add the Feed-in limatiation trough modbus and Homeassistant In solarcloud you can do it lie this This can bee usefull when prises fore the exported energy are below 0 and you dont what to export anything. |
Hello
Ok thanks. I totally missed that option.
Thanks fore a very good work.
Den ons 19 juli 2023 00:38mkaiser ***@***.***> skrev:
… already threre :)
see "export power limit"
[image: image]
<https://user-images.githubusercontent.com/29856783/254408751-ea18dff7-b6d1-4a05-b59f-35dc3b5fc427.png>
—
Reply to this email directly, view it on GitHub
<#88 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/A55DGNO5BLGMD3PCXNTR3W3XQ364ZANCNFSM6AAAAAAWPVW3JA>
.
You are receiving this because you commented.Message ID:
<mkaiser/Sungrow-SHx-Inverter-Modbus-Home-Assistant/issues/88/1641029210@
github.com>
|
@mkaiser input_select.set_sg_start_stop_mode in modbus_sungrow.yaml doesn´t exist anymore right? |
@mastameista I can affirm your suggestion. It works fine now, battery capacity is shown up the right way - important: you need to login in as fitter in isolarcloud.eu to get to advanced settings as @mastameista showed in his thread. |
renamed for more clarity to:
|
then you need to update your dashboard.yaml I think :). Sungrow-SHx-Inverter-Modbus-Home-Assistant/dashboards/_DefaultDashboard_mkaiser/dashboard.yaml Line 212 in a2d8607
|
set_sg_start_stop_mode --> set_sg_inverter_run_mode thanks @bota #88 (comment)
ooops, forgot to change that - thank you :) |
As I said: "important: you need to login in as fitter in isolarcloud.eu to get to advanced settings" from germany? then use this guidethread, howto describes how to get an fitteraccount (in case you do not have one) --> |
works great, thanks for update 💯 @mkaiser |
@btota thanks for your verification. Will also add this parameter :) |
Unfortunately, nothing changes for me even with the current yaml and multiple procedures as in the instructions - still "unavailable". What else can I do? |
Please open a new issue (this is already closed) and show us, what entities are shown in your installation, when you go to "Developer Tools" and filter entities for "battery_charge" |
In the new "default" dashboard these fields are shown as errors in the first page.
From my view the problem is that these are relicts from earlier versions. The correct entity (sensor.battery_charge_nominal) are already in the dashboard.
The text was updated successfully, but these errors were encountered: