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

Can not calibrate current scale or offset #928

Open
chris1howell opened this issue Nov 4, 2024 · 6 comments
Open

Can not calibrate current scale or offset #928

chris1howell opened this issue Nov 4, 2024 · 6 comments

Comments

@chris1howell
Copy link
Member

[Patrik Jonsson] Posted on OpenEVSE Forum...

(https://openev.freshdesk.com/users/6059459827)

I want to fix the current sensor calibration on my unit and the instructions I've found say to do this with RAPI commands, which doesn't seem to work (I can see the value but not alter it using the dev console.) Are there updated instructions for how to update the calibration without RAPI if this is no longer available? I don't see the values in the web configuration anywhere.

The RAPI commands for scale and offset are blocked but the http interface does not have the calibration UI.

@lutorm
Copy link

lutorm commented Nov 6, 2024

I realized I can update the calibration by downloading the settings json and editing it before uploading it back. Not super user friendly, but given this only needs to be done at most once it's not a big deal.

@KipK
Copy link
Collaborator

KipK commented Dec 1, 2024

Hi,

I'm currently adding this to UI.
However seems sending "scale" and "offset" value to /config doesn't works here. It still sends back default values after update.

@lutorm, do you confirm after POST the config data to /config endpoint you got scale and offset value updated thereafter ?

It's not working for me here, need to find why

@lutorm
Copy link

lutorm commented Dec 1, 2024 via email

@KipK
Copy link
Collaborator

KipK commented Dec 1, 2024

You're right. Values are updated on EVSE module, but it seems the config endpoint doesn't get latest value after updating it.

I'll fix that. thanks

@jeremypoulter
Copy link
Collaborator

FYI It might be nice to add a similar thinf to this to the UI: https://github.com/OpenEVSE/open_evse/blob/master/utils/calcalc.cpp

@KipK
Copy link
Collaborator

KipK commented Dec 1, 2024

For now, I've added it in EVSE tab
image

I'll think about what would be the best way to add this calc tool ( a new Tools sections, or a popup from the EVSE / current sensor section )

edit: @jeremypoulter doing that, we will need then to have access to the sensor raw value somehow. Not sure we have access to raw sensor data from rapi.
Can't we do it by calculating it from 2 mesured values , replacing raw by openevse reported values, and using the actual scale/offset to calculate the new ones ?
What would be the algo then ?

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

4 participants