Core: Change light color hyperion

[N1c093]

The problem

Currently it's not possible to set a light color without setting the effect manually to "Solid". If you didn't change the effect to "Solid" the color of the light doesn't change.

Is it possible to set the effect automatically to "Solid", if you change the color of the light via ui?

Here is an example where you can see, that the effect don't change to "Solid" after I change the color to green:

Environment

• Home Assistant Core release with the issue: 0.116
• Last working Home Assistant Core release (if known):
• Operating environment (OS/Container/Supervised/Core): Home Assistant OS
• Integration causing this issue: hyperion
• Link to integration documentation on our website: https://www.home-assistant.io/integrations/hyperion/

Problem-relevant configuration.yaml

light:
  - platform: hyperion
    host: XXX.XXX.XXX.XX

Traceback/Error logs

Additional information

I think it would be possible to set the effect to "Solid", when you change the color of the light. See: https://github.com/home-assistant/core/blob/ac256815f9fb4224f5b5a3bce224dc70002436e8/homeassistant/components/hyperion/light.py#L256-L265

I already tried to fix it by myself, but I didn't succeed.

[probot-home-assistant[bot]]

hyperion documentation
hyperion source
_{^{(message by IssueLinks)}}

[probot-home-assistant[bot]]

Hey there @dermotduffy, mind taking a look at this issue as its been labeled with an integration (hyperion) you are listed as a codeowner for? Thanks!
_{^{(message by CodeOwnersMention)}}

[dermotduffy]

@N1c093 thank you for the high-quality bug report. Will investigate...

[7h30n3]

Maybe it's the same issue.

But when I turn on the hyperion light and select "Solid" as effect and a color and then turn it off and back on again, the actual light doesn't turn back on as the UI suggests. I have to select a color first.

[dermotduffy]

Just an update on my side: I'm working hard on getting the config_flow for Hyperion working, including multiple & dynamic instance support, and thus being able to deprecate and ultimately remove the YAML. This is consuming all the time I have available at the moment, but once that's done -- this bug report is the next thing on my list to investigate.

[dermotduffy]

Fresh update: I have a fix for this coded, tested. I want to wait for a new Hyperion-NG build before making more changes to the HA component though, as there's a large volume of fixes on the Hyperion side coming (and until they're in a real build I can point people at, the odds of something going wrong are much higher...).