BREAKING!: moves 'NavPointPath' -> 'NavigateTo'; type x,y coords 'float -> 'qreal'

Merge Request Template

Summary

It was cooked man

NavigateTo now implemented as a standalone skill that others can include from and use. Trees wouldn't run since our proj was mixing float and QPointF, type conflicts in getting/setting blackboard ports.

Testing

  • Created tests
  • Updated tests
  • Tested in simulation (integration tests)
  • Tested in real robot (if applicable)
  • No tests (reason):

Screenshots/Videos

Related Issue(s)

Allows for progress on #613 (closed), all GSR tickets

Changes

Blast Radius

  • All GSR nodes using NavigateTo will need to readjust some port values
  • Coord types across the BT and PPMan components changed to qreal which is double
  • Added debug compiler flags if CMAKE_BUILD_TYPE is set to Debug

Checklist

  • All relevant unit tests are passing
  • Documentation updated (if applicable)
  • No debug logs or commented-out code

Reviewer Notes

Closes #461 (closed)

Edited by David Pham

Merge request reports

Loading