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

Bug(light): ATTR_COLOR_TEMP is a deprecated constant in light.py #435

Closed
Cimpel opened this issue Jan 4, 2025 · 4 comments · Fixed by #440
Closed

Bug(light): ATTR_COLOR_TEMP is a deprecated constant in light.py #435

Cimpel opened this issue Jan 4, 2025 · 4 comments · Fixed by #440
Labels
bug Something isn't working
Milestone

Comments

@Cimpel
Copy link

Cimpel commented Jan 4, 2025

HA core version

2025.1.0

Current integration version

0.6.5

Last known working integration version

0.6.5

Device type and model

Heat Pump Wi-Fi Controller 171H120F (0)

Working mobile app

SmartHome

The detail description of bug/problem

Logger: homeassistant.components.light
Bron: helpers/deprecation.py:222
integratie: Licht (documentatie, problemen)
Eerst voorgekomen: 08:49:16 (2 gebeurtenissen)
Laatst gelogd: 08:49:16

ATTR_COLOR_TEMP was used from midea_ac_lan, this is a deprecated constant which will be removed in HA Core 2026.1. Use kelvin equivalent (ATTR_COLOR_TEMP_KELVIN) instead, please report it to the author of the 'midea_ac_lan' custom integration

The logs

  1. DEBUG log MUST be enabled (Please refer to README document)
  2. Execute the action to trigger error log(you should description the error action)
  3. Upload the debug log file as attachment。
  4. Please don't paste the log content! upload as a attachment file.
  5. Please don't paste the log content! upload as a attachment file.
  6. Please don't paste the log content! upload as a attachment file.
    ...
@Cimpel Cimpel added the bug Something isn't working label Jan 4, 2025
@wuwentao
Copy link
Owner

wuwentao commented Jan 6, 2025

@Cimpel
first of all, please folllow github issue template, enable debug log and attach your debug log file.
sorry, you don't provide any detail debug log, and based on the keyword, I just search all the source code file, found it maybe include in

anyway, follow github issue template is basic requirement for bug issue.
thanks

@wuwentao wuwentao changed the title HA Core issue Bug(light): ATTR_COLOR_TEMP is a deprecated constant in light.py Jan 6, 2025
@wuwentao wuwentao added this to the 0.6.6 milestone Jan 6, 2025
@wuwentao
Copy link
Owner

wuwentao commented Jan 6, 2025

add a PR fix for it #440 , you can manual replace this file in your HA device, reboot HA, and confirm whether it can works.
especially for your device color temperature value.

thanks

manual edit detail:

  1. login to your HA device via SSH or HA web add-on
  2. cd /config/custom_components/midea_ac_lan/
  3. wget https://github.com/wuwentao/midea_ac_lan/raw/704457e975f39384af945a2c0270b330556df11c/custom_components/midea_ac_lan/light.py -O light.py
  4. reboot HA, confirm whether it can works.

@Dino0005
Copy link

Dino0005 commented Jan 7, 2025

add a PR fix for it #440 , you can manual replace this file in your HA device, reboot HA, and confirm whether it can works. especially for your device color temperature value.

thanks

manual edit detail:

  1. login to your HA device via SSH or HA web add-on
  2. cd /config/custom_components/midea_ac_lan/
  3. wget https://github.com/wuwentao/midea_ac_lan/raw/704457e975f39384af945a2c0270b330556df11c/custom_components/midea_ac_lan/light.py -O light.py
  4. reboot HA, confirm whether it can works.

me too in the logs I had the warning ATTR_COLOR_TEMP of costate deprecated.
I replaced the light.py file, after restarting Home Assistant the deprecated constant warning was no longer there.
Thanks.

@wuwentao
Copy link
Owner

wuwentao commented Jan 8, 2025

@Dino0005 thank you very much for your test result update.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants