-
Notifications
You must be signed in to change notification settings - Fork 101
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
Does not support 25Hz scanning frequency #50
Comments
Yes, I wondered about that too! |
I modified this line of code but I've obtained the following error: |
I remember vaguely trying to change this, but If my memory serves me right the object that is used to interface to the lidar breaks if one tries a different frequency. More code than this part needs to be updated. Too long ago to remember though. |
Did you succeed to make it work at the end? |
Sadly no, me and a friend dug into it and decided to build our own node to throw away un-needed measurments instead. It surely is possible but we suspect it might be easier to build the interface code from scratch if you want it to do something else... :/ So we kept ours at 50Hz. |
Ok, thank you very much 👍 |
LMS1xx/src/LMS1xx_node.cpp
Line 75 in e529960
I suggest the change to:
if (cfg.scaningFrequency != 5000 && cfg.scaningFrequency != 2500)
The text was updated successfully, but these errors were encountered: