Skip to content

997 feature choose message stream rates#1133

Merged
1Blademaster merged 12 commits intomainfrom
997-feature-choose-message-stream-rates
Mar 29, 2026
Merged

997 feature choose message stream rates#1133
1Blademaster merged 12 commits intomainfrom
997-feature-choose-message-stream-rates

Conversation

@ProgramPhantom
Copy link
Copy Markdown
Contributor

Sorry this got delayed! I've been incredibly busy over the last 7 days.

Think this should be working. I've tested it with the position data stream and it works.

…ttings are turned on.

- Populated developer tab with data stream rates fields
- Added emitter to drone connection slice
- Added emitter implementation in emitter.js
- Added button to developer tab that calls emitter
@ProgramPhantom ProgramPhantom requested a review from a team March 26, 2026 21:27
@ProgramPhantom ProgramPhantom linked an issue Mar 26, 2026 that may be closed by this pull request
@ProgramPhantom ProgramPhantom self-assigned this Mar 26, 2026
Comment thread gcs/data/default_settings.json Outdated
Comment thread gcs/src/components/settingsModal.jsx Outdated
Comment thread gcs/data/default_settings.json Outdated
Comment thread radio/app/endpoints/params.py
Comment thread radio/app/endpoints/states.py Outdated
ProgramPhantom and others added 3 commits March 27, 2026 10:58
- Changed max data stream rate to 15
- Included logging statements and stream rate validation in backend
- Formatted
@ProgramPhantom
Copy link
Copy Markdown
Contributor Author

Need more approval apparently

@1Blademaster
Copy link
Copy Markdown
Member

image

I think you accidentally missed this?

Comment thread gcs/data/default_settings.json Outdated
@ProgramPhantom
Copy link
Copy Markdown
Contributor Author

Oh yeah I don't know what happened to the max 15 commit

@ProgramPhantom
Copy link
Copy Markdown
Contributor Author

Pretty sure everything is resolved here.

@1Blademaster
Copy link
Copy Markdown
Member

Need to move the data streams from saving to a file to saving in redux. Will do in a new issue.

@1Blademaster 1Blademaster merged commit a74a130 into main Mar 29, 2026
6 of 8 checks passed
@1Blademaster 1Blademaster deleted the 997-feature-choose-message-stream-rates branch March 29, 2026 19:13
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.

[FEATURE] Choose message stream rates

2 participants