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

Error while integrating backward: Negative path velocity #8

Open
yzy244001743 opened this issue Mar 21, 2023 · 1 comment
Open

Error while integrating backward: Negative path velocity #8

yzy244001743 opened this issue Mar 21, 2023 · 1 comment

Comments

@yzy244001743
Copy link

When this happens, is there any special way to deal with it?

@BryanStuurman
Copy link

BryanStuurman commented Dec 10, 2024

I added a variable timestep function to my fork:
https://github.com/Taiga-Robotics/trajectories-totg/blob/production/src/Trajectory.cpp#L317
https://github.com/Taiga-Robotics/trajectories-totg/blob/production/src/Trajectory.cpp#L426

moveit also did something special in the same functions which I don't fully understand but also may harden the algorithm against this issue (I've never seen negative path velocity errors with moveit)
https://github.com/moveit/moveit/blob/master/moveit_core/trajectory_processing/src/time_optimal_trajectory_generation.cpp#L532
moveit/moveit#1665

Note that for the two (undocumented and lost) test cases I ran against my patch it was sufficient to fix the issue in the forward integration and the negative integration issue never popped up after that. I put the variable dt into the negative path anyways.

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