victronenergy / dynamic-ess

MIT License
85 stars 7 forks source link

[Bug]: "need a victron config client" if this is the very first victron flow added to node-red #42

Closed trooster closed 1 year ago

trooster commented 1 year ago

If this is the very first flow added to node red, the idle battery flow gives an error on the victron-idle-battery node:

"need a victron config client"

The Victron config client is only available when there is a Victron energy flow created before. After I added a flow with one of the normal Victron Energy nodes, the config client was created and the dynamic ESS flow started to work.

I don't know if this can be fixed in the ESS "victron-idle-battery" node, or that this must be added to the prerequisites in the documentation.

dirkjanfaber commented 1 year ago

Thanks for reporting. Just to make sure, you are running the latest version? IIRC I've fixed this with this commit, but I'll take a closer look at it next week. Might be that I did not include that fix in the latest release (or that it does not work all of the time).

trooster commented 1 year ago

Hi Dirkjan, I downloaded the one that was available in the latest Venus OS Large at the time I wrote the report.

dirkjanfaber commented 1 year ago

For the newer flow we remove the victron-idle-battery node, so this will no longer be an issue. There is still a bit of development time needed before that one will be released though. So closing this issue as won't fix. Work-a-round is to deploy another Victron Node first and then use the example flow.