Skip to content

Conversation

@Gold872
Copy link
Member

@Gold872 Gold872 commented May 26, 2025

Nearly every file of Autonomy has been changed in some way throughout this year, and most of the logic behind logical execution has been refactored.

Here are some of the major changes that have been made:

Project Organization

  • Reorganize folders
  • Separate configurations for drive
  • Replace SensorDrive and SensorlessDrive with one standard RoverDrive
  • Move all constants into one file

Pathfinding

  • Add diagonal movement to A*
  • Use octile distance heuristic
  • Replace all GPS related math functions with UTM coordinates

Path Following/Logic Execution

  • All path following is now synchronous with a periodic timer
  • Use state machine to handle task execution
  • Fix turning behavior to turn the shortest way
  • Use different tolerances for path following vs final destination
  • Improve abstraction and coupling between sensor vs timed + sim drive

Detector + Video

  • Incorporate lidar sensor + new video packet structure
  • Lock any found obstacles that interfere with path
  • Use path generation + solvepnp for aruco detection

Misc

  • Reformatted all files
  • Remove redundant tests

TODO:

  • Confirm state machine structure
  • Remove behavior trees
  • Finish documentation
  • Implement Aruco task with state machine
  • Trigger replan on drive timeout
  • Cleanup imports

@Gold872 Gold872 marked this pull request as ready for review October 12, 2025 18:12
@Gold872 Gold872 requested a review from lregueiferos October 12, 2025 18:12
Copy link
Contributor

@lregueiferos lregueiferos left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The autonomy rewrite looks good. I removed the commented-out files and increased the number of tries for the network rover can drive test.

@Gold872 Gold872 merged commit dbae277 into main Oct 30, 2025
2 checks passed
@Gold872 Gold872 deleted the full-rewrite branch October 30, 2025 02:49
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

Successfully merging this pull request may close these issues.

6 participants