Skip to content

Aayush/Moving-and-Dropping#24

Merged
AidenFavish merged 4 commits into
mainfrom
Aayush/Moving-and-Dropping
May 12, 2026
Merged

Aayush/Moving-and-Dropping#24
AidenFavish merged 4 commits into
mainfrom
Aayush/Moving-and-Dropping

Conversation

@aayushext

Copy link
Copy Markdown
Contributor

Implement a new MAVProtocol subclass that enables velocity-controlled waypoint navigation and triggers a payload drop at a specified waypoint. This includes logging release events for analysis and ensures constant speed at the release point.

… drop

- New MAVProtocol subclass: navigates waypoints via velocity setpoints
- Fires a drop_callback exactly once when release waypoint radius is entered
- Does NOT slow down at release point — maintains constant speed for accuracy
- Logs timestamped release events for post-flight analysis
- Registered in mavcore.protocols.__init__ for easy import
@aayushext aayushext requested a review from AidenFavish March 7, 2026 04:04
@aayushext aayushext self-assigned this Mar 7, 2026
AidenFavish
AidenFavish previously approved these changes May 12, 2026

@AidenFavish AidenFavish left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

Copilot AI review requested due to automatic review settings May 12, 2026 16:31

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review is ineligible. To be eligible to request a review, you need a paid Copilot license, or your organization must enable Copilot code review.

@AidenFavish AidenFavish left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@AidenFavish AidenFavish merged commit cb25652 into main May 12, 2026
1 check passed
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.

3 participants