Skip to content

IKEA light bulbs rename and intervals adjusted - #8112

Merged
manup merged 2 commits into
dresden-elektronik:masterfrom
mattreim:Rename-IKEA-bulbs
Feb 5, 2025
Merged

IKEA light bulbs rename and intervals adjusted#8112
manup merged 2 commits into
dresden-elektronik:masterfrom
mattreim:Rename-IKEA-bulbs

Conversation

@mattreim

Copy link
Copy Markdown
Collaborator

Change:

  • tradfribulbe14wscandleopal470lm.json -> tradfri_bulb_e14_ws_candle_opal_470lm.json
  • tradfribulbe27wsglobeopal1055lm.json -> tradfri_bulb_e27_ws_candle_opal_1055lm.json

@mattreim

mattreim commented Feb 1, 2025

Copy link
Copy Markdown
Collaborator Author

Hello everyone, I have a question about these two setting options for the ws bulbs, which one is better?

A

        {
          "name": "cap/color/capabilities",
          "refresh.interval": 86400  [or "static": 16]
        },
        {
          "name": "cap/color/ct/max",
          "static": 454
        },
        {
          "name": "cap/color/ct/min",
          "static": 250
        },

B

        {
          "name": "cap/color/capabilities",
          "refresh.interval": 86400
        },
        {
          "name": "cap/color/ct/max",
          "refresh.interval": 86400,
          "default": "0xfeff"
        },
        {
          "name": "cap/color/ct/min",
          "refresh.interval": 86400,
          "default": 0
        },

@manup

manup commented Feb 3, 2025

Copy link
Copy Markdown
Member

Good question, imho both are ok. The static one safes a bit over the air traffic and expects that no other version of this model with different settings exists (which likely is true in 99% of cases). The second one is more generic in kinda future proof.

@manup manup added this to the v2.29.2-beta milestone Feb 3, 2025
@mattreim

mattreim commented Feb 3, 2025

Copy link
Copy Markdown
Collaborator Author

Should I switch the other bulbs to version b?

@manup

manup commented Feb 5, 2025

Copy link
Copy Markdown
Member

I think it's ok as is for now.

@manup
manup merged commit b38846b into dresden-elektronik:master Feb 5, 2025
@mattreim

mattreim commented Feb 5, 2025

Copy link
Copy Markdown
Collaborator Author

@manup Now you can close this old PR #7480

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants