Skip to content

DDF for iluminize motion sensor (5128.10) #7046

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

Merged

Conversation

AndreKoepke
Copy link
Contributor

See #7045

@AndreKoepke
Copy link
Contributor Author

I tried it with in my local home. This ddf seems to be working (setting lift/tilt works without any problems).

It was the first time, that I wrote a ddf. I just copied it and changed the adresses.

@BabaIsYou
Copy link
Contributor

Did not really understand binding on cluster "0x0102" "cl": "0x0102" on endpoint 6 "src.ep": 6, because device descibes on referenced issue doesn't have endpoint 6

@AndreKoepke
Copy link
Contributor Author

@BabaIsYou
I am very new to DDF and I guessed the values. But from your comment I learned that "cl" stands for cluster. :D
Feel free to edit. I can test it with my devices.

@BabaIsYou
Copy link
Contributor

Think you should have a look to https://github.com/dresden-elektronik/deconz-rest-plugin/wiki/DDF-cheat-sheet then would understand what I meant ;-)

@AndreKoepke
Copy link
Contributor Author

I will read it and fix the JSON. ^^
It turns out that after a while, that the value-report don't works as expected. :/

@BabaIsYou
Copy link
Contributor

BabaIsYou commented Jun 12, 2023

As the device is quite simple, perhaps it would be better to start from scratch using DDF editor and compare the final DDF with the one you did ;-)

but I think that essentially it’s around the wrong EP … it’s not 6 but 1 if I’m referring to your issue (#7045) .

@AndreKoepke
Copy link
Contributor Author

:D
I tried to write the plain JSON while there is a DDF editor. omg.

I fix the JSON and I will change it here, when the tests are successful and my slow train-vpn-forwarded-x11-ssh-connection allows my to save.

AndreKoepke and others added 3 commits June 12, 2023 18:09
May be some bindings will be missing for reports
@BabaIsYou
Copy link
Contributor

I did the changes from EP 6 to EP 1 into the DDF but you have to test before merging and close #7045

@AndreKoepke
Copy link
Contributor Author

AndreKoepke commented Jun 12, 2023

Ok. I dropped the settings via DDF-editor and add it from the default-values. Maybe the endpoint came from there.

On the first view, it seems to be working. I will report again, when I'm back home.
Thanks for the change. :)

image

@AndreKoepke
Copy link
Contributor Author

AndreKoepke commented Jun 12, 2023

I'm at home and I had to fix the state/open. But all seems to work now.

@BabaIsYou Thanks for the help and the hints. It helped a lot. :)

Suppress path, useless description and replace ep:0 by ep:1
Fix dresden-elektronik#7045
@AndreKoepke
Copy link
Contributor Author

Seems good. ready4merge from my site.

@AndreKoepke AndreKoepke mentioned this pull request Jul 9, 2023
@Mimiix Mimiix linked an issue Jul 9, 2023 that may be closed by this pull request
@manup
Copy link
Member

manup commented Jul 20, 2023

@BabaIsYou all good to merge?

@BabaIsYou
Copy link
Contributor

@BabaIsYou all good to merge?

Looks good at this step for me.

@manup manup merged commit f180342 into dresden-elektronik:master Jul 20, 2023
@manup manup added this to the v2.23.0-beta milestone Jul 20, 2023
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.

Iluminize 5128.10
3 participants