UserID and Charging Max Amps

[casi23]

Hello and thank you for this great extension. Now I finally feel able to run my river2 without the EcoFlow Cloud!
2 mistakes I had when setting up:

1. When using the username and password, there was an error to which I should read the log. Unfortunately, there was nothing in the log. With the use of the user ID, the river2 can be integrated.
2. DC Charging Max Amps, shows me the current installation. When changing the value, the process is detected by the river2 (display goes on), but not executed.

Greetings
Carsten

[GnoX]

1. Are you sure you didn't just not scroll high enough? Logs are a bit weird in that if you search for this integration, it will only filter visible logs, not all logs, you have to actually load them for filter to show anything. Nothing we can do without the error messages.
2. Well, I know there are several different versions that set this for similar Delta 2 devices, but I don't think I've seen more versions of this control in the code, I'll recheck to be sure. Does the DC mode select work by chance?

[casi23]

Good morning.

The DC mode is working. The switchover is handled by the River.

[GnoX]

Oh, there was actually a bug there, I think you're the first one to actually test that slider. Try if it works in v0.42.0-g.1 from my fork.

[jor93]

Hello @GnoX
I updated to the latest version (I skipped a few versions) and now I am also having issues updating the Charging Max Amps for the Delta 3. This worked before withouth any issue. Trying to update the value gives following error (same behavior from the device page):

Error from HA log:

Logger: homeassistant.helpers.script.websocket_api_script
Source: helpers/script.py:531
First occurred: 23:22:02 (2 occurrences)
Last logged: 23:22:04

websocket_api script: Error executing script. Unexpected error for call_service at pos 1: '>' not supported between instances of 'float' and 'NoneType'
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/helpers/script.py", line 531, in _async_step
await getattr(self, handler)()
File "/usr/src/homeassistant/homeassistant/helpers/script.py", line 1018, in _async_step_call_service
response_data = await self._async_run_long_action(
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
...<9 lines>...
)
^
File "/usr/src/homeassistant/homeassistant/helpers/script.py", line 631, in _async_run_long_action
return await long_task
^^^^^^^^^^^^^^^
File "/usr/src/homeassistant/homeassistant/core.py", line 2817, in async_call
response_data = await coro
^^^^^^^^^^
File "/usr/src/homeassistant/homeassistant/core.py", line 2860, in _execute_service
return await target(service_call)
^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/src/homeassistant/homeassistant/helpers/service.py", line 835, in entity_service_call
single_response = await _handle_entity_call(
^^^^^^^^^^^^^^^^^^^^^^^^^^
hass, entity, func, data, call.context
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
)
^
File "/usr/src/homeassistant/homeassistant/helpers/service.py", line 907, in _handle_entity_call
result = await task
^^^^^^^^^^
File "/usr/src/homeassistant/homeassistant/components/number/init.py", line 105, in async_set_value
if value < entity.min_value or value > entity.max_value:
^^^^^^^^^^^^^^^^^^^^^^^^
TypeError: '>' not supported between instances of 'float' and 'NoneType'

[casi23]

yes, in v0.42.0-g.1 "Charging Max Amps" works