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

Rituals Perfume Genie Can't control status on/off #135012

Open
Zaleo80 opened this issue Jan 7, 2025 · 2 comments
Open

Rituals Perfume Genie Can't control status on/off #135012

Zaleo80 opened this issue Jan 7, 2025 · 2 comments

Comments

@Zaleo80
Copy link

Zaleo80 commented Jan 7, 2025

The problem

I can see all the data and status.
But when I try to switch the device on/off the following error is shown. Device state does not seem to change.

Screenshot_20250107-173129

What version of Home Assistant Core has the issue?

core-2025.1.1

What was the last working version of Home Assistant Core?

No response

What type of installation are you running?

Home Assistant OS

Integration causing the issue

rituals_perfume_genie

Link to integration documentation on our website

https://www.home-assistant.io/integrations/rituals_perfume_genie/

Diagnostics information

No response

Example YAML snippet

No response

Anything in the logs that might be useful for us?

Logger: homeassistant
Bron: components/rituals_perfume_genie/select.py:36
Eerst voorgekomen: 12:04:28 (108 gebeurtenissen)
Laatst gelogd: 17:36:52

Error doing job: Task exception was never retrieved (None)
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 1552, in _async_process_registry_update_or_remove
    self.async_write_ha_state()
    ~~~~~~~~~~~~~~~~~~~~~~~~~^^
  File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 1023, in async_write_ha_state
    self._async_write_ha_state()
    ~~~~~~~~~~~~~~~~~~~~~~~~~~^^
  File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 1148, in _async_write_ha_state
    self.__async_calculate_state()
    ~~~~~~~~~~~~~~~~~~~~~~~~~~~~^^
  File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 1085, in __async_calculate_state
    state = self._stringify_state(available)
  File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 1029, in _stringify_state
    if (state := self.state) is None:
                 ^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/components/select/__init__.py", line 145, in state
    current_option = self.current_option
                     ^^^^^^^^^^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/components/rituals_perfume_genie/select.py", line 80, in current_option
    return self.entity_description.current_fn(self.coordinator.diffuser)
           ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/components/rituals_perfume_genie/select.py", line 36, in <lambda>
    current_fn=lambda diffuser: str(diffuser.room_size_square_meter),
                                    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.13/site-packages/pyrituals/__init__.py", line 106, in room_size_square_meter
    }[self.room_size]
      ^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.13/site-packages/pyrituals/__init__.py", line 96, in room_size
    return int(self.hub_data["attributes"]["roomc"])
ValueError: invalid literal for int() with base 10: ''

Additional information

Using the v2 genie, in the original App I can control the device state.

@home-assistant
Copy link

home-assistant bot commented Jan 7, 2025

Hey there @milanmeu, @frenck, mind taking a look at this issue as it has been labeled with an integration (rituals_perfume_genie) you are listed as a code owner for? Thanks!

Code owner commands

Code owners of rituals_perfume_genie can trigger bot actions by commenting:

  • @home-assistant close Closes the issue.
  • @home-assistant rename Awesome new title Renames the issue.
  • @home-assistant reopen Reopen the issue.
  • @home-assistant unassign rituals_perfume_genie Removes the current integration label and assignees on the issue, add the integration domain after the command.
  • @home-assistant add-label needs-more-information Add a label (needs-more-information, problem in dependency, problem in custom component) to the issue.
  • @home-assistant remove-label needs-more-information Remove a label (needs-more-information, problem in dependency, problem in custom component) on the issue.

(message by CodeOwnersMention)


rituals_perfume_genie documentation
rituals_perfume_genie source
(message by IssueLinks)

@tusmum
Copy link

tusmum commented Jan 7, 2025

Same here

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants