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

Improve DDF for dresden elektronik's Kobold #7696

Closed
wants to merge 2 commits into from
Closed

Improve DDF for dresden elektronik's Kobold #7696

wants to merge 2 commits into from

Conversation

ebaauw
Copy link
Collaborator

@ebaauw ebaauw commented Apr 7, 2024

  • Add missing capabilities and config items;
  • Use attribute reporting for state/on and state/bri - any reason why this was disabled in favour of polling?

@ebaauw ebaauw requested a review from manup April 7, 2024 10:11
@ebaauw ebaauw added Device Improvement Additional tag to attach to a existing issue. Enhancement labels Apr 7, 2024
@ebaauw ebaauw added this to the v2.27.0-beta milestone Apr 7, 2024
@ebaauw ebaauw changed the title Improve DDF for dresden elektronik's kobolt Improve DDF for dresden elektronik's kobold Apr 7, 2024
@ebaauw ebaauw changed the title Improve DDF for dresden elektronik's kobold Improve DDF for dresden elektronik's Kobold Apr 7, 2024
@ebaauw
Copy link
Collaborator Author

ebaauw commented Apr 8, 2024

Hm, seeing this every five minutes, and by the looks of it, also on change reports:

 Warning duplicated zcl.cmd: 0x0A, cluster: 0x0006 with zcl.seq: 57 for dresden elektronik / Kobold

The zcl.seq value changes between messages, not sure what is duplicated. Do I need to sniff the traffic, or is this a known issue?

Kobold was on firmware 0102.10000026, but I see no changes after upgrading to .1000002A. Apart from the warnings in the deCONZ log, attribute reporting seems to work, and the GUI attributes as well as the API resource items are updated.

@manup
Copy link
Member

manup commented May 13, 2024

Thanks for looking into this 👍 It's a tricky one, for now I'd like to keep the disabled reporting since I need to make some more testing first how this works in different setups.

Iirc the disabled reporting is there due the "non coordinator / lost coordinator" setups which the device needs to support.
For example if a Kobold is setup via deCONZ to control a group of lights, and afterwards the coordinator is turned off. The unicast bindings are disabled so that there isn't endless traffic towards the coordinator in that case. Technically not the best approach but that was the path with limited development time available ;)

Copy link
Contributor

github-actions bot commented Jun 1, 2024

Hey @ebaauw, thanks for your pull request!

Tip

Modified bundles can be downloaded here.
Relative expire date

DDF Bundles changes

Modified

  • dresden_elektronik/kobold.json : Kobold ✔️

Validation

Tip

Everything is fine !

🕟 Updated for commit 3957500

@ebaauw ebaauw closed this by deleting the head repository Aug 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Device Improvement Additional tag to attach to a existing issue. Enhancement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants