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

secondary info not showing data in kw + kw_decimals #288

Closed
dr3amr opened this issue Jun 16, 2023 · 8 comments · Fixed by #296
Closed

secondary info not showing data in kw + kw_decimals #288

dr3amr opened this issue Jun 16, 2023 · 8 comments · Fixed by #296
Labels
bug Something isn't working

Comments

@dr3amr
Copy link

dr3amr commented Jun 16, 2023

I have a power-flow-card-plus that shows values only in KW for day (same as the energy dashboard).
Bellow details show 0kW for the secondary sensor.solar_self_use_day even though the value of the sensor is already in kWh. If changing the unit to "kW", the value changes but with value is without decimals 5.51 on sensor would be displayed as 6 not taking account of kw_decimals value.

kw_decimals: 1
watt_threshold: 0
solar:
entity: sensor.on_off_grid_day
secondary_info:
entity: sensor.solar_self_use_day
display_zero: true

@dr3amr dr3amr added the bug Something isn't working label Jun 16, 2023
@flixlix
Copy link
Owner

flixlix commented Jun 16, 2023

This is expected if you are overriding the unit of measurement. Please configure the decimals option if you want to change the number of decimals

@flixlix flixlix closed this as completed Jun 16, 2023
@dr3amr
Copy link
Author

dr3amr commented Jun 16, 2023

kw_decimals set to any value (changes for all other values, grid solar etx) but secondary info value remains same without decimals. If we are rushing to close issues before properly understanding them...means this is just lack of proper willing to support...

@flixlix
Copy link
Owner

flixlix commented Jun 16, 2023

I just don't think you are reading what I wrote.
I said that if you want to set the decimals for the secondary info, use the decimals option in secondary info.
If you need help configuring, let me know. I just don't consider this a bug...

@dr3amr
Copy link
Author

dr3amr commented Jun 17, 2023

decimals option only available for individual not for secundary info. It does nothing when added to secondary as per bellow:

solar:
entity: sensor.on_off_grid_day
secondary_info:
entity: sensor.solar_self_use_day
unit_of_measurement: "kW"
decimals: 1
display_zero: true

@flixlix flixlix reopened this Jun 18, 2023
@flixlix flixlix linked a pull request Jun 19, 2023 that will close this issue
@flixlix
Copy link
Owner

flixlix commented Jun 19, 2023

Sorry for the inconvenience, I didn't understand the issue at first 😅
This is indeed a bug, opened #296
Should be fixed in the next version

@flixlix
Copy link
Owner

flixlix commented Jun 26, 2023

Fixed this issue in version 0.1.6.1 🥳
Thank you for submitting the issue

@flixlix flixlix closed this as completed Jun 26, 2023
@dr3amr
Copy link
Author

dr3amr commented Jun 30, 2023

Please add the new attribute also to Readme.md => Secondary info

@flixlix
Copy link
Owner

flixlix commented Jun 30, 2023

Added to the Readme :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants