search

lincomatic / open_evse

Firmware for Open EVSE
GNU General Public License v3.0
114 stars 163 forks source link

RAPI command to set voltage #121

Closed jeremypoulter closed 4 years ago

jeremypoulter commented 4 years ago

For the solar divert we are adding config to allow setting the voltage to either a static value or an MQTT feed from an energy monitor for live updates. This should allow for better conversions from power to amps.

It would also make sense to also feed this into the EVSE so it can better calculate the power.

lincomatic commented 4 years ago

Does this need to be a non-volatile value, or can it just reload the default at each boot?

glynhudson commented 4 years ago

Volatile is fine, and preferable IMO. I think most users using PV divert would choose to have VRMS set via a real time MQTT feed since it's a requirement for PV diver to have an energy monitor which will also be monitoring VRMS

lincomatic commented 4 years ago

https://github.com/lincomatic/open_evse/commit/b1b60491d38e389c1135d79944d1e163954b8c0d

jeffl012 commented 4 years ago

I see this thread is closed, but would it be possible to add a manual setting similar to the "SC amps [V] - set current capacity" command? This way a user could hard set the voltage one time for their environment (ie: 208 or 240) in lieu of obtaining dynamic energy monitor readings and obtain more accurate kWh output.

glynhudson commented 4 years ago

Thanks @lincomatic the $SV rapi command works well. Is there any chance you could append a version number to this change?

lincomatic commented 4 years ago

I see this thread is closed, but would it be possible to add a manual setting similar to the "SC amps [V] - set current capacity" command? This way a user could hard set the voltage one time for their environment (ie: 208 or 240) in lieu of obtaining dynamic energy monitor readings and obtain more accurate kWh output.

you can open that up as an issue if you like and i'll see if i can do it when i'm less busy. big problem is, we're nearly out of flash. there might not be any space to implement that

lincomatic commented 4 years ago

Thanks @lincomatic the $SV rapi command works well. Is there any chance you could append a version number to this change?

you mean RAPI version? Sure, I can increment it, but I've kind of moved away from that, because it's very hard for an applications programmer to keep track of what version of RAPI does what. Personally, I just try sending a command, and if it barfs, I know it isn't supported.