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

Feedback #1

Open
lienbacher opened this issue May 31, 2023 · 3 comments
Open

Feedback #1

lienbacher opened this issue May 31, 2023 · 3 comments

Comments

@lienbacher
Copy link

lienbacher commented May 31, 2023

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!

@thefipster
Copy link
Owner

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!

@stress1ner
Copy link

stress1ner commented Aug 27, 2023

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?

@sumit-bajaj
Copy link

Thanks for sharing your work. I managed to get the MR60BHA1 working using your code.

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

4 participants