parapluplu
parapluplu
Hey, 1. Yes, it's actually kW. 2. The graph shows the `ActivePower` channel of the ess, so `SymmetricEss.ChannelId.ACTIVE_POWER`, which is the active power of the device. 3. Partially yes. I...
I've reproduced the problem on a simulated system now to generate graphs for the fix in this merge request (and not just for my previous hotfix). The graphs are much...
Hey @huseyinsaht, thanks for looking into that. I've a few clarification questions: > We have reviewed your code and noticed that you are manipulating the input of the PID controller....
@TheSerapher Your examples are not related to this pull request. This pull request is about changing the active power of the ESS from some positive/negative value to 0 and don't...
We're running this implementation now in production since the beginning of the year and it still solves the problem. At the moment I don't understand completely what's the blocker for...
@Laksh26 Could you please provide me access to the branch? Then I would continue to add the desired changes to this pull request
@Laksh26 nice, I have push access, that should be sufficient. Thank you! Next week I'll likely find some time to work on this. @sebastianasen That's only helpful to checkout this...
I've reworked the controller a bit so it states more explicitly to support a parallel multi use scenario. I'm still not happy with the configuration (minSocLimit and socHysteresis are always...
Very interesting and nice pull request! Do you have benchmarks values?
Could somebody explain to me we codecov/patch is failing with a 50% diff target? I've basically removed code and added tests. The contrary should happen and 50% is waaaaay out...