Current shaper module

[KipK]

Current Shaper Class WIP

configuration:
mqtt_live_pwr : mqtt topic for instantaneous power taken from the grid
current_shaper_max_pwr : max power you can get from your energy plan in Watts

Fixes #65

[jeremypoulter]

Does the max power need to be a feed? Would it not be a fixed value?

[KipK]

Does the max power need to be a feed? Would it not be a fixed value?

not really for my use case.
I've kept it dynamic just in case some countries has different energy plan depending of season, time and co.
This info is already provided by my counter and I use it on other scripts so I use the mqtt feed to not have to change the value at different places.

[KipK]

latest commit seems functional. I've tested it for few hours, from another branch merged with my mqtt mods.
There's no config yet from the web ui, but works with the api.

[KipK]

I think we have everything needed to do the web UI parts now.

[jeremypoulter]

Code changes look good, will give it a test

[KipK]

I've added the UI part. There's a pull request on the gui repo too: OpenEVSE/openevse_wifi_gui#79
Probably something to change in config_save_current_shaper() in app_config.cpp, line 422.
I had to force config_changed(), haven't understood how it should be done properly.

[KipK]

I don't get why it says it has conflicts with the static files here.
Does it need the gui branch to be merged first on your side?

[KipK]

Code changes look good, will give it a test

you can test booth mqtt mods and shaper merged on this branch if needed :
https://github.com/KipK/ESP32_WiFi_V4.x/tree/mqtt_shaping

[jeremypoulter]

There have just been some updates to the GUI so you need to merge master on both the GUI and main firmware repos

[KipK]

updates to the GUI so you need to merge master on both the GUI a

missed those, thanks.

[KipK]

should be ok now

[KipK]

you need first to get the gui pull request OpenEVSE/openevse_wifi_gui#79 to make it compile on the CI/CD

[jeremypoulter]

Hiya, can you merge master into this and the GUI? I will fix the GUI ref after merging this PR

[KipK]

Seems I've messed up with my git. I need to cleanup

[jeremypoulter]

Yeah, was about to say something has gone very wrong ;-)

[jeremypoulter]

I don't think we need additional APIs for enabling/disabling the shaper, this can entirely be done via the config. To my mind it is a safety feature so should be setup and enabled then left. I don't think it should have the same MQTT/HTTP APIs to 'easily' switch on/off from external sources that the divert mode does, but other than that I think this is looking really good, will do some testing , hopefully tomorrow.

[KipK]

Looks good now

[jeremypoulter]

Ok fair enough.

…

On Fri, Jul 29, 2022, 7:07 PM Guillaume S ***@***.***> wrote: Looks good now I don't think we need additional APIs for enabling/disabling the shaper, this can entirely be done via the config. To my mind it is a safety feature so should be setup and enabled then left. I don't think it should have the same MQTT/HTTP APIs to 'easily' switch on/off from external sources that the divert mode does, but other than that I think this is looking really good, will do some testing , hopefully tomorrow. I have some use case where it's needed. Sometime I override the shaper to let the car having max priority on my house load. Then the house heaters are shaping the current instead. Therefore, I've made a quick live switch to override it without writing in the config. — Reply to this email directly, view it on GitHub <#382 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ABOVNAEF7JZTH4V25Q6FEA3VWQMWBANCNFSM5ZUCJURA> . You are receiving this because you commented.Message ID: ***@***.***>

[jeremypoulter]

I have tested this and it looks to work well. I did find a bug in the EVSE Manager when the current sharper was the only claim, and have pushed a fix.