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

Cannot compile #1

Open
fonitzu opened this issue May 1, 2020 · 5 comments
Open

Cannot compile #1

fonitzu opened this issue May 1, 2020 · 5 comments

Comments

@fonitzu
Copy link

fonitzu commented May 1, 2020

Hi,
I cannot found a suitable CircularBuffer.h

Thanks,
Florin

@d00616
Copy link
Owner

d00616 commented May 2, 2020

Hello Florin,

there is an Arduino-Library in the library manager.

Regards,

Frank

@fonitzu
Copy link
Author

fonitzu commented May 5, 2020

Hello Frank,

I was finally able to find something out but had to change your code.

The actual, real problem, is with nRF52's library in MySensors.
There are some issues, see
mysensors/MySensors#1413
mysensors/MySensors#1410
mysensors/MySensors#1371

I have tried to figure out what your code does, and found some issues.
Maybe you can find some time to take a look at:
https://github.com/fvpetrov/MySensors/tree/issue/1410-nrf5-Deadlock
in file hal/transport/NRF5_ESB/driver/Radio_ESB.cpp

I have added my remarks as compiler warnings.
I would really appreciate if you can find some time for it.

Thank you!
Florin

@d00616
Copy link
Owner

d00616 commented May 5, 2020

Hello Florin,

thank you for debugging. I hope you can fix the ESB implementation. I you fave questions please ask, I try to help but my time for this is limited at the moment.

@fonitzu
Copy link
Author

fonitzu commented May 5, 2020

Hello Frank,

Thanks for your help!

I would say that the most productive way to move things further would be a phone call, when both of us are in front of the (same) code. 10 to 15 minutes at most. According to my knowledge, my Berlin is in Germany ;), so it would be quite easy.

Otherwise, I can ask questions, but will take longer.

I only need to understand some things in code, to be able to debug it further.

@d00616
Copy link
Owner

d00616 commented May 9, 2020

Hello Florin,

Please contact me via E-Mail. One of my mail-adresses is on the MySensors About Page.

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

2 participants