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

System Losses sensor may be giving incorrect values #33

Open
StealthChesnut opened this issue Aug 5, 2022 · 5 comments
Open

System Losses sensor may be giving incorrect values #33

StealthChesnut opened this issue Aug 5, 2022 · 5 comments
Assignees
Labels
bug Something isn't working

Comments

@StealthChesnut
Copy link
Owner

StealthChesnut commented Aug 5, 2022

The system losses sensor using LAN config appears to be reading more like a current power generation figure, not a losses figure.
In the USB config it may not be accurate or is giving some surprisingly high figures.

Units may be wrong, or source data may be wrong (incorrect register definition in the LAN config?)

Need to check data source registers are correct in LAN config (or at least match the USB config), then check the implementation of the sensor template.

@StealthChesnut StealthChesnut added the bug Something isn't working label Aug 5, 2022
@StealthChesnut StealthChesnut self-assigned this Aug 5, 2022
@StealthChesnut
Copy link
Owner Author

Likely related to Discussion #9

@StealthChesnut StealthChesnut changed the title System Losses sensor giving incorrect values (LAN) System Losses sensor may be giving incorrect values Aug 5, 2022
@TonyM1958
Copy link

Don't know if this helps, I changed how system_losses are calculated in my setup, using RPower 31008. More details in the code snippet here: https://foxesscommunity.com/viewtopic.php?p=907&sid=9a6298b7f5e2a6a928328633d103530a#p907

@StealthChesnut
Copy link
Owner Author

Thanks Tony - appreciate the contribution. I'm not sure how we'd verify how accurate any approach is. Any thoughts @Daibutt ?

@TonyM1958
Copy link

I've added a comment to the disucssion on the community forum linked above, as the calculations depend on some sensor name changes in your code. I made these changes for consistency between USB and LAN and also to align more with Nathan Marlor. May be worth having a look to see if you want to incorporate any of the changes into the next release?

Ideally, I think the various names would be resolved by adding consistent unique_ids to the entities, so this is something I am also looking into.

@TonyM1958
Copy link

oops - looked at the code I posted and realised I hadn't actually copied the code I use for System Losses! Corrected this now.

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

No branches or pull requests

2 participants