Testers needed for Netatmo climate (multiple thermostats or valves)

Hi,

My experience with this component using: 1 thermostat + 8 valves is that works fine.

There is a comment in GitHub about tabulation bug in line +75 in netatmo2.py file, that should be considered to get all netatmo valves in HA.

Thanks @gieljnssns !!! You should add this module to HA with a pull request …

I’ve changed my gist…

hey,

I was with mmornati (the one that made the comment on github) playing with your script when we found the tabulation error…by fixing this you re now adding all entities in your script but unfortunately all the informations attached to the first entity added, is added to all entities.

Plus there is bit of interesting info in the API to get (boiler_valve_comfort_boost, battery, …)…we gonna try to take a look later also

Thx anyway for the job done. I really feel you re not far from getting done !

I’d like to confirm that, I can see all my valves but all get the same values from the wirst entity. Also controlling one of the valves only controls the first one.

1 Like

That’s the same behavior of multiple thermostat

So, is this Component dead already?

I have the same problems as everyone. I see my 2 Valves (Name), but the attributes are bothh from the first one (ID, Room ID, Temperatures). So it’s pretty useless at the moment:(

yes, unless somebody that knows how to code, is going to fix it

I’m also waiting on this…:slight_smile:

#Metoo :smiley:

Hi,

I think i found the problem.

In line 69 you get a list of roomdata.
In line 70 you have a loop at room_names, but not on room data.
so in line 75 you add room_names of the loop and room_data, which isn’t looped. Thats why only the data of the first entry is shown.
If you change the loop in 70 to room_data, so that you loop on every single room, and you add a line where you get the room_name out of the looped room_data, than it should work.

2 Likes

Care to share the fixed code? I get what you are saying but I lack any python skills to fix it.

I am sure is something (relatively) simple. I can’t code, but I have 2 thermostat and I can check if it works, if you can instruct me how to put the corrected code

Can you try this version? Just download the file and put it under custom_components/climate. See the first post for detailed instructions.

1 Like

I will try it later, but at the first look, the only diffrence is, that the if was changed and the add_devices is inside this if.
But the problem i assumed is still the same, only the room_names are looped not the room_data.

2018-11-20 09:12:51 ERROR (MainThread) [homeassistant.components.climate] Error while setting up platform netatmo2
Traceback (most recent call last):
File “/usr/local/lib/python3.6/site-packages/homeassistant/helpers/entity_platform.py”, line 128, in _async_setup_platform
SLOW_SETUP_MAX_WAIT, loop=hass.loop)
File “/usr/local/lib/python3.6/asyncio/tasks.py”, line 358, in wait_for
return fut.result()
File “/usr/local/lib/python3.6/concurrent/futures/thread.py”, line 56, in run
result = self.fn(*self.args, **self.kwargs)
File “/config/custom_components/climate/netatmo2.py”, line 70, in setup_platform
for room_name in room_data.get_room_names():
File “/config/custom_components/climate/netatmo2.py”, line 314, in get_room_names
self.setup()
File “/config/custom_components/climate/netatmo2.py”, line 329, in setup
self.homestatus = lnetatmo.HomeStatus(self.auth, home=self.home)
File “/config/deps/lib/python3.6/site-packages/netatmo_home/Thermostat.py”, line 125, in init
self.rawData = resp[‘body’][‘home’]
KeyError: ‘home’

Tested your version but all the test-devices disappeared (restarted twice to make sure). Reverted to the previous version and they got back.

I still can control only one

Thanks for the feedbar. I’ll take a closer look. Unfortunately I don’t have Netatmo valves myself. Which previous version exactly?

The version I downloaded yesterday from @gieljnssns

1 Like

https://gist.github.com/gieljnssns/4ff4d0d5548c614c548de8b77dcd11d2 <- I assume?