Uh oh!
There was an error while loading. Please reload this page.
new changes - #2
Conversation
Caution Review failedThe pull request is closed. 📝 WalkthroughWalkthroughThe PR refactors the robot architecture from direct KronBot usage to a Robot singleton pattern, integrates PedroPathing for autonomous navigation via a Follower object, introduces a new Heading subsystem for pose filtering, and updates all autonomous and manual operations to use centralized robot control mechanisms. Changes
Sequence DiagramsequenceDiagram
participant Init as Initialization
participant Robot as Robot Singleton
participant Follower as PedroPathing Follower
participant Heading as Heading Subsystem
participant Subsystems as Subsystems (Intake, Turret, etc.)
Init->>Robot: getInstance()
Init->>Robot: initHardware(hardwareMap)
Robot->>Follower: create & configure
Init->>Robot: initFollower(hardwareMap, pose)
Robot->>Follower: setPose()
loop Loop Cycle
Follower->>Follower: update pose
Follower-->>Robot: getCurrentPose()
Robot->>Heading: update(rawHeading)
Heading->>Heading: apply delta filter
Heading-->>Robot: get() filtered heading
Robot->>Subsystems: update with filtered heading
Robot->>Follower: setTeleOpDrive() / followPath()
end
Estimated Code Review Effort🎯 4 (Complex) | ⏱️ ~45 minutes Poem
✨ Finishing Touches
🧪 Generate unit tests (beta)
Tip Issue Planner is now in beta. Read the docs and try it out! Share your feedback on Discord. Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Before issuing a pull request, please see the contributing page.
Summary by CodeRabbit
Release Notes
New Features
Improvements