-
Notifications
You must be signed in to change notification settings - Fork 11
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
Add another manufacturer PID #100
Add another manufacturer PID #100
Conversation
Thank you very much @peternewman , i am getting through my mail bag. I have emailed Tomas (our main RDM developer) for full list of our manufacturer specific PIDs, hopefully to have something soon. cheers Petr |
That's great thanks @vanous . In the meantime are you happy with this one, or would you rather I wait until we've got the full list from your developer? |
Hi Peter, i have spoken with our dev and he will look at this after his holidays (2wks, starting today). So if this is no blocker, please wait and we merge it together. |
No blocker, I'm just being impatient, but it certainly makes sense to wait until your actual RDM dev is available, that would be perfect. |
oh, speak of impatient :) i have been pushing him for this, but he is very good, thus very busy... but once he promises, he usually delivers :) |
Hi Peter, terribly swamped here. Nonetheless, finally, have some PIDs :) Cheers Petr ROBE_PT_FEED 0xa000 Pan/Tilt Feedback 0-off 1-on |
Perfect thanks @vanous . I don't mind doing some of the leg work on the formatting of the PIDs if you'd like? Certainly for the boolean ones, it should be a fairly simple copy and paste job. Do those PIDs all support get and set? Do any of your bits of kit have subdevices, and if so, can get and set be targeted at root or sub devices for the various PIDs? Do you refer to them as ROBE_ in the manual, we seem to be about 50/50 currently with previous PIDs as to whether they're prefixed with the manufacturer name or not. Finally what do you want to do about the existing PR? Are you happy with it, shall we merge it? |
@peternewman i am waiting for Tomas to reply :) |
Hi @vanous , Did Tomas ever reply to you? 😄 I'm not quite clear, does "Other then that i am happy with the PR." mean we need his reply to merge this PR? Or was that just regarding the other PIDs? |
Hi Peter, i have missed your email and am very sorry about it. The PIDs are OK, please merge. Thank you Petr |
Blocked behind #196 . |
From the snapshot here:
OpenLightingProject/ola#1275
To be reviewed by @vanous .