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

'Backup reserve' option missing on D2M #137

Closed
walderston opened this issue Aug 26, 2024 · 20 comments
Closed

'Backup reserve' option missing on D2M #137

walderston opened this issue Aug 26, 2024 · 20 comments

Comments

@walderston
Copy link

Using ecoflow-mqtt via HA and noticed the 'Backup reserve' switch / option is not shown. If I use the hassio-ecoflow-cloud via HACS the option is there.

What do you need from me to look at getting this added?

@foxthefox
Copy link
Owner

I am not sure what is the data point behind "backup reserve"
Do you mean pd.newAcAutoOnCfg or pd.bpPowerSoc ?
Both are available and can be found in the iobroker object tree and should also be shown in HA.

If you refer to something else, I need more information.

@foxthefox
Copy link
Owner

pd.watchIsConfig is available in the adapter as "diagnostic" data.
I did not know that this is a switching datapoint.

Can you please enable the debug mode in the adapter and additionally

  • the debug of the D2M in the configuration line (tick in box)
  • the msgSet/Get (tick in box)

After debug mode is working, please perform this command in the APP and post the log file with some pre- and following lines.

@walderston
Copy link
Author

walderston commented Aug 29, 2024

Sure 2 separate logs files for turning the switch on vs off:
[backup on.log]
[backup off.log]

@foxthefox
Copy link
Owner

Unfortunately the log does not show the command from the APP. it only shows some unidentified values, which might come from a newer FW.
I think before of repeating the test, please check the enable status of the device and and general debug settings.
I have some exemplary screenshots:

Bildschirmfoto 2024-09-01 um 18 40 46

Bildschirmfoto 2024-09-01 um 18 41 52

Bildschirmfoto 2024-09-02 um 08 03 06

If these checkmarks are set, then the log should have more info inside (and will also report the missing datapoints) and should also have the command given by the APP

@walderston
Copy link
Author

walderston commented Sep 2, 2024

ah sorry, didn't realise I had to enable debug in the device config too.

Hopefully I timed that right as its very chatty - I filtered on 0077 being the last digits of the serial number to reduce the amount of chatter.

off.log
on.log

@foxthefox
Copy link
Owner

Thanks for the log.
There are new data structures inside, which fail to be processed. This is handled, but some important data updates are not visible.
I will prepare a quick fix for that tomorrow.
After upgrade unfortunately the test needs to becrepeated.
I’ll keep you informed when I am ready.

ps.
There are a lot of new datapoints inside the telegram, which need to be defined. That comes later.

@foxthefox
Copy link
Owner

I have not finished the next version 1.0.3, but the branch is uploaded.
You can already try to create a new on.log and off.log with that version.

You need to go to adapters-> octacat ->custum and type the url as in the screenshot

Bildschirmfoto 2024-09-03 um 07 02 24

@walderston
Copy link
Author

walderston commented Sep 3, 2024

updated and attached:
backup off.log
backup on.log

@foxthefox
Copy link
Owner

Thanks a lot, now it is clear how it is arranged.
I will update accordingly.

@foxthefox
Copy link
Owner

You could have a try, once again with the "tree" version (still 1.0.3)
You have to delete the datapoint pd.watchIsConfig and pd.bpPowerSoc or the whole pd-tree and restart the adapter.
That creates the new datapoints including their new behaviour.

pd.watchIsConfig is disabling/anabling the backup mode
pd.bpPowerSoc is changing the value and when changing the value it is enabling the mode (not sure if the APP does it in the same way)

@walderston
Copy link
Author

seems to be working as expected. thanks very much!

@foxthefox
Copy link
Owner

OK, I will make it official.

Can you check if the issue with SlowChangeWatt is still there, it might be a result of the not expected telegrams.

@walderston
Copy link
Author

yup thats not updating in HA when I update the value in HA. It does update the value within the app itself.

@walderston
Copy link
Author

hopefully this has the right debugging on.

SlowChangeWatt.txt

Basically, change the SlowChgWatts value in HA to 1800, that value it sent to EF however nothing is sent back to HA to set the new value to 1800

So when I refresh the HA page, it contains the old value.

@foxthefox
Copy link
Owner

foxthefox commented Sep 5, 2024

Very good, you discovered the tick for HA debugging.
What I am missing in this log is the [set_reply] message.

Maybe there is some behaviour which I do not anticipate.
It could be the case that the HA always needs a response to the changed value and therefore it is not memorizing the given command.
In iobroker a msg could contain the Ack=false and true to distinguish between command and feedback, but a given command is stored and not artificially reset (due to missing acknowledge).

I am so far not aware of a bit in HA to give feedback/acknowledgment or the necessity of such.

The adapter is only sending updated values to HA if there is a change to the current value or a complete update after adapter start.

@foxthefox
Copy link
Owner

I could also see it at my implementation, after the page refresh the value is back to the previous one.

@walderston
Copy link
Author

going to assume its the same thing as permanentWatts that I mentioned on the last comment of #143

not tested the other values but I'd imagine its the same behaviour that HA keeps the old value rather than the newly set value so when you refresh in HA, its now out of sync until either ioBroker adapter is restarted or some other unknown event.

@foxthefox
Copy link
Owner

foxthefox commented Sep 5, 2024

I would assume it in the same way. But still it is strange for me that a command is not persistent stored. At least it is processed as such and was going through the different processes. 🤔

@foxthefox
Copy link
Owner

Maybe we should continue in the other issue, hence this one is solved.

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

2 participants