-
Notifications
You must be signed in to change notification settings - Fork 18
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
also receive commands #6
Comments
Hi 'thewhobox', |
@Bill-Gaytes yes I got it working. See pending pull request. |
Hi Mike,
Firstly thanks you for your reply. I will admit that I don't have a great
deal of experience with the workings of GitHub so that's why I didn't look
in Pull requests.
I can see the Pull request where you made changes to Dali.cpp and then 3
Commits to make changes to Dali.h, DaliBus.cpp and DaliBus.h ?
Would I be correct that I now need to take your changes and update the Dali
library ?
I have all the Dali hardware for this but not really certain with the
software. The example (Dali Blink) is not really that helpful either.
Sorry for my naivety, but we all have keep learning.
Cheers Bill
…On Sat, Oct 21, 2023 at 9:32 PM Mike ***@***.***> wrote:
@Bill-Gaytes <https://github.com/Bill-Gaytes> yes I got it working. See
pending pull request.
—
Reply to this email directly, view it on GitHub
<#6 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AW7XMGUUNNQJDAEOE7CAZYTYAQWM7AVCNFSM6AAAAAAZYLTH6CVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONZTHEYTKMRRGM>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
Hey, there are two options to get it work in your project.
DaliBus.receivedCallback([](uint8_t *data, uint8_t len) -> void {
//do stuff with the raw message here
});
but you will have to parse these messages by yourself. |
Hi Mike,
Many thanks, I followed the branch and updated the library, looks to be OK.
Other than the dali_blink sketch, do you know of any other examples using
this library that I can have a look at so I can learn a bit more ?
Cheers
…On Sun, Oct 22, 2023 at 11:29 AM Mike ***@***.***> wrote:
Hey, there are two options to get it work in your project.
- you can make the changes manually by copy the changes in DaliBus.cpp
and DaliBus.h and then set
DaliBus.receivedCallback([](uint8_t *data, uint8_t len) -> void {
//do stuff with the raw message here
});
- use my fork branch:
https://github.com/thewhobox/arduino-dali/tree/feature-add-receiving
but you will have to parse these messages by yourself.
—
Reply to this email directly, view it on GitHub
<#6 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AW7XMGXPQM6TMQWIL6PU7G3YATYSHAVCNFSM6AAAAAAZYLTH6CVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONZUGA2TMNBZG4>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
Depends on what you want to learn. You would also need to implement memory banks, if you want your device to be setup by Dali Cockpit or something else: |
Hi @Bill-Gaytes! |
Is it possible to enhance this lib to receive commands from the Dali bus?
With this we could build a dali monitor or even own dali devices.
I think a good startpoint could be:
arduino-dali/src/DaliBus.cpp
Lines 207 to 208 in 28aafd0
If we dont expect a response it is a command.
The text was updated successfully, but these errors were encountered: