-
Notifications
You must be signed in to change notification settings - Fork 9
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
Feedback #1
Comments
Hi lienbacher, thanks for your feedback. I will definitely check out the PR and see what I can do better regarding the UART pins. So far they are changable via Substitutions, I thougt this was the main way to do it as far as I have seen in other repos, but you always learn something new ;) I will also have a go at the restart problem stopping the uart communication. Although if I remember correctly that didn't happen with my MCUs, so hopefully I can recreate the scenario. Thanks again! |
Hi, thanks for the great work. After flashing the proximity sensor does not seem to work is there any thing I need to do to make it work? |
Thanks for sharing your work. I managed to get the MR60BHA1 working using your code. |
Hey Fipster!
I really like the work you did here! Well done, thank you very much!
Unfortunately it seems on an esp32-s2 it crashes after a few hours, but I was not able to aquire any logs to find out what the the reason could be.(Edit: it was an issue with the oscillator on my S2 mini, not related to your component)I have also found that after restarting the esp with the mmwave sensor (MR24HPC1) on it would not be able to communicate anymore. Wild guessing it might be the uart becoming out of sync.
One more thing regarding configuration, the UART pins are fixed in the package. I'd actually configure the uart outside of the package.
You may wanna have a look at this pr in esphome: esphome/esphome#4434
This is a very nicely made component for the LD2410 mmwave sensor. I guess there will be a lot of similarities with the ones you integrated, maybe it will help you improve your implementation.
Unfortunately I am full with projects at the moment, in case that changes I might just go ahead and try myself. Sounds like a fun venture.
Thanks for your work!
The text was updated successfully, but these errors were encountered: