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

Feedback #24

Open
woopstar opened this issue Nov 14, 2024 · 39 comments
Open

Feedback #24

woopstar opened this issue Nov 14, 2024 · 39 comments
Assignees
Labels
documentation Improvements or additions to documentation help wanted Extra attention is needed skip-changelog

Comments

@woopstar
Copy link
Owner

Report bugs and/or feedback in relation to the energy sensors in this topic.

@Extati
Copy link

Extati commented Nov 15, 2024

After latest Update, working mode sensor is now unknown / ukendt - making the program not working anymore?

@woopstar
Copy link
Owner Author

#23 should fix it. I will merge and release it.

@Extati
Copy link

Extati commented Nov 15, 2024

Perfect - Thanks

@Extati
Copy link

Extati commented Nov 15, 2024

Another request. Right now I dont have an EV charger sensor, our any export sensor. It looks like when it cant find the sensor, it keep filling the log about a missing sensor every second or so, so it build up the .db way to fast. Is there a fix for that?

@woopstar woopstar self-assigned this Nov 15, 2024
@woopstar woopstar added documentation Improvements or additions to documentation skip-changelog labels Nov 15, 2024
@woopstar
Copy link
Owner Author

Another request. Right now I dont have an EV charger sensor, our any export sensor. It looks like when it cant find the sensor, it keep filling the log about a missing sensor every second or so, so it build up the .db way to fast. Is there a fix for that?

Can you create a new issue and put some of the logs into it, so I can see what output it is and surpress it.

@woopstar woopstar changed the title Home Assistant Energy Sensors Feedback: Home Assistant Energy Sensors Nov 15, 2024
@woopstar woopstar pinned this issue Nov 15, 2024
@Extati
Copy link

Extati commented Nov 15, 2024

Another request. Right now I dont have an EV charger sensor, our any export sensor. It looks like when it cant find the sensor, it keep filling the log about a missing sensor every second or so, so it build up the .db way to fast. Is there a fix for that?

Can you create a new issue and put some of the logs into it, so I can see what output it is and surpress it.

Sure - Will do it later

@Extati
Copy link

Extati commented Nov 15, 2024

#23 should fix it. I will merge and release it.

Just updated, restartet and configed again, still shows working sensor as Unknown / Ukendt

@woopstar
Copy link
Owner Author

#23 should fix it. I will merge and release it.

Just updated, restartet and configed again, still shows working sensor as Unknown / Ukendt

Can you provide me with logs and a screenshot of the attributes on the working mode sensor:

image

@Extati
Copy link

Extati commented Nov 15, 2024

What logs do you wanna see? The working sensor is empty.
FYI: I use the standard names from Huawei solar intergration.
Also my net consumption is: Unknown / Ukendt ?

Hope you can see something from this:

last_updated: "2024-11-15T18:22:53.397980"
last_changed_mode: "2024-11-15T18:21:53.393172"
unique_id: hsem_workingmode_sensor
huawei_solar_device_id_inverter_1_id: fe8976f680ef7c70154d4c4a472ad13a
huawei_solar_device_id_inverter_2_id: null
huawei_solar_device_id_batteries_id: 79013076495028544b2c0de5ef408d2f
huawei_solar_batteries_working_mode_entity: select.batteries_working_mode
huawei_solar_batteries_working_mode_state: time_of_use_luna2000
huawei_solar_batteries_state_of_capacity_entity: sensor.batteries_state_of_capacity
huawei_solar_batteries_state_of_capacity_state: 5
huawei_solar_batteries_grid_charge_cutoff_soc_entity: number.batteries_grid_charge_cutoff_soc
huawei_solar_batteries_grid_charge_cutoff_soc_state: 100
huawei_solar_batteries_maximum_charging_power_entity: number.batteries_maximum_charging_power
huawei_solar_batteries_maximum_charging_power_state: 2500
huawei_solar_inverter_active_power_control_state_entity: sensor.inverter_efficiency
huawei_solar_inverter_active_power_control_state_state: "100.0"
huawei_solar_batteries_tou_charging_and_discharging_periods_entity: sensor.batteries_time_of_use_periods
huawei_solar_batteries_tou_charging_and_discharging_periods_state: "2"
huawei_solar_batteries_tou_charging_and_discharging_periods_periods:

  • 06:00-10:00/1234567/-
  • 17:00-23:59/1234567/-
    house_consumption_power_entity: sensor.house_power_consumption
    house_consumption_power_state: 240
    solar_production_power_entity: sensor.inverter_input_power
    solar_production_power_state: 0
    net_consumption: null
    net_consumption_with_ev: 0
    energi_data_service_import_entity: sensor.energi_data_service
    energi_data_service_import_state: 2.553
    energi_data_service_export_entity: sensor.energi_data_service_produktion
    energi_data_service_export_value: 0.573
    battery_max_capacity: 5
    battery_remaining_charge: 4.75
    battery_conversion_loss: 10
    ev_charger_status_entity: sensor.go_echarger_is_charging
    ev_charger_status_state: null
    ev_charger_power_entity: sensor.go_echarger_222819_nrg_12
    ev_charger_power_state: null
    house_power_includes_ev_charger_power: true
    morning_energy_need: 1.5
    solcast_pv_forecast_forecast_today_entity: sensor.solcast_pv_forecast_forecast_today
Skærmbillede 2024-11-15 kl  18 25 12 Skærmbillede 2024-11-15 kl  18 25 28

@Extati
Copy link

Extati commented Nov 15, 2024

In System logs i also get this error now after the update.

Skærmbillede 2024-11-15 kl  20 19 20 Skærmbillede 2024-11-15 kl  20 19 32

@woopstar
Copy link
Owner Author

What logs do you wanna see? The working sensor is empty. FYI: I use the standard names from Huawei solar intergration. Also my net consumption is: Unknown / Ukendt ?

Hope you can see something from this:

It did. Net consumption was not calculated because you don't have a charger. If no net consumption the sensor does not work. Can you update to 1.0.2 and see if it fixes the issue.

@Extati
Copy link

Extati commented Nov 15, 2024

It did. Net consumption was not calculated because you don't have a charger. If no net consumption the sensor does not work. Can you update to 1.0.2 and see if it fixes the issue.

I have an EV Charger, but it is a Clever, so can't get it in, before I get my Shelly 3EM.
Now I can't get pass the EV charger setup.

Skærmbillede 2024-11-15 kl  21 00 58

@woopstar
Copy link
Owner Author

It did. Net consumption was not calculated because you don't have a charger. If no net consumption the sensor does not work. Can you update to 1.0.2 and see if it fixes the issue.

I have an EV Charger, but it is a Clever, so can't get it in, before I get my Shelly 3EM. Now I can't get pass the EV charger setup.

Skærmbillede 2024-11-15 kl 21 00 58

can you try the latest release 1.0.3. It should be fixed now.

@Extati
Copy link

Extati commented Nov 16, 2024

can you try the latest release 1.0.3. It should be fixed now.

Now I got past it, and looks correct whit the working sensor now.

Only thing now, is I'm not sure I use the right sensor for "Huawei solar inverter active power control state entity" We don't have any sensor named that whit the standard config from Huawei solar integration. We have "sensor.inverter_efficiency" - I assume that is the correct one?

Thanks again for a very quick fix.

@woopstar
Copy link
Owner Author

can you try the latest release 1.0.3. It should be fixed now.

Now I got past it, and looks correct whit the working sensor now.

Only thing now, is I'm not sure I use the right sensor for "Huawei solar inverter active power control state entity" We don't have any sensor named that whit the standard config from Huawei solar integration. We have "sensor.inverter_efficiency" - I assume that is the correct one?

Thanks again for a very quick fix.

The sensor sensor.inverter_active_power_control is disabled by default in the Huawei integration. You need to to the the Huawei integration and click the entities that are hidden for the inverter. Then enable the entity by clicking it.

@Extati
Copy link

Extati commented Nov 16, 2024

The sensor sensor.inverter_active_power_control is disabled by default in the Huawei integration. You need to to the the Huawei integration and click the entities that are hidden for the inverter. Then enable the entity by clicking it.

Ah okay, I saw it yesterday, but didn't think that it was that. After your picture yours says limited to 100%, where mine says unlimited. I have now activated it, and make the hole setup again.

Again, thank you so much for the effort you put in this.
Also appreciate the quick replies.

@Extati
Copy link

Extati commented Nov 26, 2024

Not sure if you want me to open a issue or not, but it looks like we don't have sensor.batteries_rated_capacity under inverter or batteries that can be enabled. Only rated_capacity is under the PEES intergration, but that is from a input and not as a sensor.

Can you see were you have sensor.batteries_rated_capacity from evt.?

@woopstar
Copy link
Owner Author

Not sure if you want me to open a issue or not, but it looks like we don't have sensor.batteries_rated_capacity under inverter or batteries that can be enabled. Only rated_capacity is under the PEES intergration, but that is from a input and not as a sensor.

Can you see were you have sensor.batteries_rated_capacity from evt.?

I have it under Diagnostics for Batteries:

image

@woopstar
Copy link
Owner Author

I'm running 1.5.0a1 of Huawei integration

@Extati
Copy link

Extati commented Nov 26, 2024

I'm running 1.5.0a1 of Huawei integration

Ah, must be the reason then. On Public release 1.4.1 we don't have that option.
Maybe you can add, so both old, and new method for SOC can be used?

@woopstar
Copy link
Owner Author

I'm running 1.5.0a1 of Huawei integration

Ah, must be the reason then. On Public release 1.4.1 we don't have that option. Maybe you can add, so both old, and new method for SOC can be used?

Let me check when 1.5.0 will be out. I think it's soon. Rather use the option.

Until then, we could create an input helper type Number, and set it to 0 to x.000 Watts. So if you have 10kWh battery, set it to 10.000

Would that work?

@Extati
Copy link

Extati commented Nov 26, 2024

Yes, that should work in theory.
Else you chould make it so it also could pull the data from the "Huawei_solar_pees" intergration, it uses the sensor called:

input_number.battery_rated_capacity

Don't know what makes the most sense to use.

@woopstar
Copy link
Owner Author

Yes, that should work in theory. Else you chould make it so it also could pull the data from the "Huawei_solar_pees" intergration, it uses the sensor called:

input_number.battery_rated_capacity

Don't know what makes the most sense to use.

v1.2.2 should fix this and allow input_number for the sensor.

@Extati
Copy link

Extati commented Nov 26, 2024

v1.2.2 should fix this and allow input_number for the sensor.

Great work as always. Will test later and let you know if I have any errors.

@Extati
Copy link

Extati commented Nov 26, 2024

Great work as always. Will test later and let you know if I have any errors.

In v1.2.2, I again can't get passed the EV charger setup-page, I only have a sensor for the EV status sensor, and not for the power sensor. When I try to set it up again, I get:

Entity is neither a valid entity ID nor a valid UUID

The error also comes if you don't enter anything.

@woopstar
Copy link
Owner Author

In v1.2.2, I again can't get passed the EV charger setup-page, I only have a sensor for the EV status sensor, and not for the power sensor. When I try to set it up again, I get:

Entity is neither a valid entity ID nor a valid UUID

The error also comes if you don't enter anything.

Can you try v1.2.3 now

@Extati
Copy link

Extati commented Nov 26, 2024

Can you try v1.2.3 now

Yes, now I got pass the setup. The only noticeable thing under the working mode is this:

Batteries conversion loss: Ukendt
Batteries remaining charge: Ukendt

That is not right, is it?

Edit: Also the "Huawei solar batteries rated capacity state" sensor is: Ukendt

My helper is: input_number and measure number is Wh like yours is on the attached picture from your sensor in diagnostic.

Also "Huawei solar batteries tou charging and discharging periods periods" is now : 00:00-00:01/1234567/+
Is that right now?

@woopstar
Copy link
Owner Author

The sensor sensor.inverter_active_power_control is disabled by default in the Huawei integration. You need to to the the Huawei integration and click the entities that are hidden for the inverter. Then enable the entity by clicking it.

Ah okay, I saw it yesterday, but didn't think that it was that. After your picture yours says limited to 100%, where mine says unlimited. I have now activated it, and make the hole setup again.

Again, thank you so much for the effort you put in this.

Also appreciate the quick replies.

Can you screenshot the configuration screens and also all attributes ? Might take a minute or two for the sensor to adapt. But the rated capacity is missing.

@Extati
Copy link

Extati commented Nov 26, 2024

Can you screenshot the configuration screens and also all attributes ? Might take a minute or two for the sensor to adapt. But the rated capacity is missing.

The sensor has not updated yet, I think it's the input helper not giving the data it's needed.

Screenshots of the sensors is here:

Skærmbillede 2024-11-26 kl  20 29 33 Skærmbillede 2024-11-26 kl  20 29 43 Skærmbillede 2024-11-26 kl  20 30 01

And helper is like this:

Skærmbillede 2024-11-26 kl  20 34 34

Looks like its not using the right input sensor for the batteries rated capacity. Try to just configure HSEM. Don't delete the device and re-add it. But configure.

The sensor is set to sensor. And you are using a input_number. So it looks like its using the wrong entity here.

Can't configure again, once it has been setup, can't get pass the EV charger again, as I'm missing one sensor. So are getting the same error as before your fix. Was the same in the other updates, so I have to delete, and restart after every update ;-)

Edit: Just tried to use another sensor for the EV charger to get pass it, and set it up again, seams that you can't change the sensor to an input sensor in this version. So I think the error is there around that area somehow.

@Extati
Copy link

Extati commented Nov 26, 2024

Edit: Just tried to use another sensor for the EV charger to get pass it, and set it up again, seams that you can't change the sensor to and input. So I think the error is there somewere.

Back on v1.1.1 again, and all sensors and work mode is fine again. Will wait until further updates.

@woopstar
Copy link
Owner Author

Back on v1.1.1 again, and all sensors and work mode is fine again. Will wait until further updates.

I found the issue and will soon release 1.2.4, which fixes it. Thank you for testing everything!

@Extati
Copy link

Extati commented Nov 27, 2024

I found the issue and will soon release 1.2.4, which fixes it. Thank you for testing everything!

Great work. Sure, it's a pleasure to test, and see the work you are implementing, to make it better all the time.

@Extati
Copy link

Extati commented Nov 27, 2024

I found the issue and will soon release 1.2.4, which fixes it. Thank you for testing everything!

Updated to 1.2.4. and now it look more right. The only thing that confuses me is the Batteries conversion loss, is still shows as: Ukendt

Also the "Huawei solar batteries tou charging and discharging periods periods" shows: 00:00-00:01/1234567/+ and not the time like in the old versions. Don't know if this is intended or not?

See below sensor settings now in .1.2.4:

Skærmbillede 2024-11-27 kl  17 46 21 Skærmbillede 2024-11-27 kl  17 46 32 Skærmbillede 2024-11-27 kl  17 46 58

@woopstar
Copy link
Owner Author

Updated to 1.2.4. and now it look more right. The only thing that confuses me is the Batteries conversion loss, is still shows as: Ukendt

Will be fixed in #66 - I will release 1.2.5 in a moment to fix it.

The TOU mode is OK. It should change later on.

@Extati
Copy link

Extati commented Nov 27, 2024

The TOU mode is OK. It should change later on.

Indeed it did - so far all seams good - I would say 1.2.5 is stable and working perfect so far.

@woopstar woopstar changed the title Feedback: Home Assistant Energy Sensors Feedback Dec 6, 2024
@woopstar woopstar added the help wanted Extra attention is needed label Dec 6, 2024
@Extati
Copy link

Extati commented Dec 11, 2024

Just for everyone to know, version 2.0.0 is super great, stabe and charging/discharging is so far spot on.
Great work as always.

@Extati
Copy link

Extati commented Dec 18, 2024

Update 2.1.0 have a fault when you update from a working intergratin you get an error saying that you are missing inputs, when you try to config, you again can't get pass the EV charger page..

@woopstar
Copy link
Owner Author

Update 2.1.0 have a fault when you update from a working intergratin you get an error saying that you are missing inputs, when you try to config, you again can't get pass the EV charger page..

2.1.1 should fix the issue. Just published it.

@Extati
Copy link

Extati commented Dec 18, 2024

2.1.1 should fix the issue. Just published it.

Indeed it did - Thanks again for the quick fix as always.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation help wanted Extra attention is needed skip-changelog
Projects
None yet
Development

No branches or pull requests

2 participants