behavior tree plugin PathLongerOnApproach bug report about navigation2 HOT 5 CLOSED

@StetroF are you open to submitting the PR to resolve 😄 I'd love to have you contribute your own patch in!

Sure! It's my first time to pull request. I will have try
#4344

from navigation2.

Hey @StetroF

Thanks for reporting the issue. I guess you are using one of the smac planners if I'm not wrong. This plugin was always tested with the NavFn planner while I developed it. I oversaw this issue, since NavFn planner for some reason has the same timestamps.

The solution for this is simply straightforward, would you be interested in sending us a PR? I will be happy to review it for you.

from navigation2.

That seems sensible - just change to old_path.poses.back().pose == new_path.poses.back().pose

from navigation2.

@StetroF are you open to submitting the PR to resolve 😄 I'd love to have you contribute your own patch in!

from navigation2.

Thanks for the fix @StetroF :-)

from navigation2.