-
Notifications
You must be signed in to change notification settings - Fork 18k
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
Rover: bendyruler does not resume original route even if OA_OPTIONS=0 #20406
Comments
I don't think scurve navigaiton has more advantages than L1 method in Boat. Besides, it has better turning performance. Moreover, the turning performance depends on very large deceleration. From my test, the L1 method is sufficient in terms of track mapping accuracy, obstacle avoidance effect and automatic control under high flow rate, at least for the current master branch. |
@xianglunkai, thanks very much for testing and reporting issues with the new SCurves on Rover. We've still got time to fix any issues before 4.3 and I would very much like it if we can convince you to use SCurves because I really want to avoid two different versions. I guess the picture above is not of master Rover? I guess instead this is a modified version that gives up on the current waypoint and moves to the next one if it sees an obstacle? I'm just trying to understand how I could reproduce the issue you're seeing and help resolve it. |
@rmackay9 Thank you very much. You are right. I use the modified version of master, but even if I use master directly, there is still the problem that the route cannot be restored after obstacle avoidance, which is unbearable for surveying and mapping. |
Finally, whether we could design a trajectory input interface in guide mode for the navigation needs of ROS or other systems. Maybe I could give some code for the development of obstacle avoidance and planning system, but I still need your help. |
Re the issue of the Rover not returning to the original path, you've tired setting AVOID_OPTIONS = 0 of course? |
@rmackay9 Yes ,I set OPTIONS = 0,but it does not work at all |
OK, thanks. Do you think the issue is new to 4.3? |
@rmackay9 Yes,I am sure this pr is new for current master |
@rmackay9
Looking forward to discussion, if feasible, I would like to submit a fix. |
The text was updated successfully, but these errors were encountered: