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

zigbee-herdsman migration #23

Closed
hobbyquaker opened this issue Sep 6, 2019 · 14 comments
Closed

zigbee-herdsman migration #23

hobbyquaker opened this issue Sep 6, 2019 · 14 comments

Comments

@hobbyquaker
Copy link
Contributor

hobbyquaker commented Sep 6, 2019

Hi Koenkk! First of all: Thanks for your awesome work!
I have questions about the herdsman:

  • will you move functionality like handling reporting, configure, deviceAvailability (and other stuff that is currently implemented in zigbee2mqtt as extension) to zigbee-herdsman?
  • I'm already using zigbee-herdsman@0.1.17 in node-red-contrib-zigbee, worked flawlessly as a drop-in replacement for the shepherd. As far as I saw you introduced some breaking changes in the higher versions of the herdsman - will you create a little "migration guide" pointing out the API changes?

Regards,
Sebastian

Ref: hobbyquaker/node-red-contrib-zigbee#44

@Koenkk
Copy link
Owner

Koenkk commented Sep 6, 2019

Hey Sebastian!

@hobbyquaker
Copy link
Contributor Author

Thanks for the answer and the API hints! Will try to move my project to 0.6 asap ;)

@Koenkk
Copy link
Owner

Koenkk commented Sep 6, 2019

Great, feel free to ask any questions here.

@hobbyquaker
Copy link
Contributor Author

hobbyquaker commented Sep 12, 2019

@Koenkk I would like to offer my help with documentation: adding jsdoc comments in the source, configure jsdoc&jsdoc2markdown to create an API.md file documenting all methods/properties/events. What do you think about that? Would you accept PRs?

@Koenkk
Copy link
Owner

Koenkk commented Sep 12, 2019

Yes definitely! Let me know if something is unclear.

I would recommend starting with the things I mentioned here: #23 (comment)

@hobbyquaker
Copy link
Contributor Author

hobbyquaker commented Sep 12, 2019

Have to get some more jsdoc(2md) knowledge, this is the first time I'm using it on Typescript and with Async/Await/Promises... This is just a first example so you get some impression on how this could look like ;-)

@Koenkk
Copy link
Owner

Koenkk commented Sep 12, 2019

That looks like a good start 😄

@hobbyquaker
Copy link
Contributor Author

Hi @Koenkk
I managed it to adapt node-red-contrib-zigbee to zigbee-herdsman@0.7.0 and on my test system everything runs fine. I'm very excited about the herdsman, pairing works much better, until now everything works stable and consuming the api is much more pleasant than working with the shepherd :-) Thank you so much for your work!

Now i tried to upgrade my production system, but I was facing a problem. There are no messages incoming from my sensors, trying to control bulbs always ends in no network route (205). Do I see that right that this migration should just work (starting with already paired devices and a device db that was created by the shepherd)?

@hobbyquaker
Copy link
Contributor Author

ah, nevermind, got it working 🎉
Made a little mistake leading to another panID: hobbyquaker/node-red-contrib-zigbee@8588311 :-)

@Koenkk
Copy link
Owner

Koenkk commented Sep 27, 2019

Good to hear 😄

@hobbyquaker
Copy link
Contributor Author

Another Question: I'm keeping track of binds in the device.meta to offer a nice ui for binding:
image
What do you think about the suggestion to integrate that mechanism in the herdsman? Right now binds made by the configure extensions aren't tracked because it seemed to complicated to hook oh these, that's something I'd like to have in future... If you want I could try to prepare a PR, but most likely I will need some hands-holding and pushes into the right direction ;-)

@hobbyquaker
Copy link
Contributor Author

hobbyquaker commented Sep 27, 2019

And one more question: I also have a ui to select which devices should report:
image
But I have no clue how to "un-report". Removing the bind isn't enough I guess, can I remove them completely by some special configuration on endpoint.configureReporting(cluster, configuration)?

@Koenkk
Copy link
Owner

Koenkk commented Sep 27, 2019

@bohtho
Copy link

bohtho commented Nov 14, 2019

This is exemplary complementary development! And with such quality! I’m ditching my current zigbee system and migrating it all to zigbee-herdsman/node-red-contrib-node-zigbee based on reputation, documentation and what I see here alone.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants