Skip to content

Update valid baud rates and remove deprecated values#1017

Merged
1Blademaster merged 2 commits intomainfrom
1014-feature-allow-more-baud-rates-on-connection
Feb 28, 2026
Merged

Update valid baud rates and remove deprecated values#1017
1Blademaster merged 2 commits intomainfrom
1014-feature-allow-more-baud-rates-on-connection

Conversation

@1Blademaster
Copy link
Copy Markdown
Member

No description provided.

Copilot AI review requested due to automatic review settings February 26, 2026 20:38
@1Blademaster 1Blademaster linked an issue Feb 26, 2026 that may be closed by this pull request
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the list of valid baud rates used for drone communication by removing deprecated values (300 and 250000) and adding support for higher modern baud rates (460800, 500000, 625000, 921600, 1000000, 1500000). Additionally, it fixes a typo in the JSX file where "74880" was incorrectly listed instead of "75880".

Changes:

  • Removed deprecated baud rates 300 and 250000 from both backend and frontend
  • Added six new higher baud rates (460800, 500000, 625000, 921600, 1000000, 1500000) to support faster communication speeds
  • Fixed typo in frontend: corrected "74880" to "75880" to match the backend value

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
radio/app/drone.py Updated VALID_BAUDRATES constant by removing deprecated rates and adding six new higher baud rates
gcs/src/components/navbar.jsx Updated baud rate dropdown options to match backend changes and fixed typo (74880 -> 75880)

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread radio/app/drone.py
Comment thread gcs/src/components/navbar.jsx
@1Blademaster 1Blademaster requested a review from a team February 26, 2026 21:10
@1Blademaster 1Blademaster merged commit 1700780 into main Feb 28, 2026
8 of 9 checks passed
@1Blademaster 1Blademaster deleted the 1014-feature-allow-more-baud-rates-on-connection branch February 28, 2026 05:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[FEATURE] Allow more baud rates on connection

3 participants