Confusion entre hvac_mode et hvac_action dans climate.py about hilo

[julienld]

Merci pour le release!

[DBestman]

Tu as très bien décrit le même problème que j'ai observé, et je suis d'accord avec tes observations. Je t'encourage à le régler! 😁

[valleedelisle]

Belle observation, je crois également que tu as raison après avoir lu la doc.

Je crois qu'on devrait mettre la logique dans python-hilo comme hvac_mode et tout simplement exposer hvac_mode dans l’intégration home assistant.

Si j'ai bien compris:

• On remplacerait hvac_mode ici par hvac_action et on remplace heat par heating et off par idle, a moins qu'il soit offline.

https://github.com/dvd-dev/python-hilo/blob/main/pyhilo/device/climate.py#L31-L34

• hvac_mode retournerait tout simplement toujours heat, à moins que vous vouliez vraiment un off à 5C? Je ne vois pas de state offline:
  https://github.com/dvd-dev/python-hilo/blob/main/pyhilo/device/climate.py#L31-L34

• On ajoute un hvac_action ici un peu comme le hvac_mode:
  

  hilo/custom_components/hilo/climate.py

  Lines 85 to 87 in 6e96be3

  	@property
  	def hvac_mode(self):
  	return self._device.hvac_mode

Malheureusement, Hilo ne semble pas supporté les calls de type set_hvac_mode (ex pour mettre a off), donc ici, ça reste pareille:

hilo/custom_components/hilo/climate.py

Lines 81 to 83 in 6e96be3

	def set_hvac_mode(self, hvac_mode):
	"""Set operation mode."""
	return

Et tant qu'à travailler là dedans, il faudrait presque migrer vers les Enum car les constantes sont dépréciées.
https://github.com/home-assistant/core/blob/dev/homeassistant/components/climate/const.py#L8-L31

J'ai toujours pas environnement de dev et j'manque de temps pour deepdive tout ça, mais si vous soumettez des PR, je vais les reviews et merged.

[julienld]

Je vais essayer de faire une PR en fin de semaine, mais je ne garantis rien.

[ic-dev21]

Belle observation, je crois également que tu as raison après avoir lu la doc.

Je crois qu'on devrait mettre la logique dans python-hilo comme hvac_mode et tout simplement exposer hvac_mode dans l’intégration home assistant.

Si j'ai bien compris:

• On remplacerait hvac_mode ici par hvac_action et on remplace heat par heating et off par idle, a moins qu'il soit offline.

https://github.com/dvd-dev/python-hilo/blob/main/pyhilo/device/climate.py#L31-L34

• hvac_mode retournerait tout simplement toujours heat, à moins que vous vouliez vraiment un off à 5C? Je ne vois pas de state offline:
  https://github.com/dvd-dev/python-hilo/blob/main/pyhilo/device/climate.py#L31-L34
• On ajoute un hvac_action ici un peu comme le hvac_mode:
  

  hilo/custom_components/hilo/climate.py

  Lines 85 to 87 in 6e96be3

  	@property
  	def hvac_mode(self):
  	return self._device.hvac_mode

Malheureusement, Hilo ne semble pas supporté les calls de type set_hvac_mode (ex pour mettre a off), donc ici, ça reste pareille:

hilo/custom_components/hilo/climate.py

Lines 81 to 83 in 6e96be3

	def set_hvac_mode(self, hvac_mode):
	"""Set operation mode."""
	return

Et tant qu'à travailler là dedans, il faudrait presque migrer vers les Enum car les constantes sont dépréciées. https://github.com/home-assistant/core/blob/dev/homeassistant/components/climate/const.py#L8-L31

J'ai toujours pas environnement de dev et j'manque de temps pour deepdive tout ça, mais si vous soumettez des PR, je vais les reviews et merged.

Je comprends pas mal où tu t'en vas, le bout qui m'échappe c'est où pyhilo entre en ligne de compte? Il ne fait pas parti du custom component, est-il installé via core? Où se trouvent ces fichiers si je veux tenter l'expérience de mon bord?

[julienld]

J'ai fait les 2 pull requests... testé avec HA 2023.2.3.

dvd-dev/python-hilo#75
#220

[valleedelisle]

Merci bien pour les commits et pour avoir tester. J'vais faire une release plus tard ce weekend!