-
Notifications
You must be signed in to change notification settings - Fork 18k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
SITL: add ELRS simulator #26663
Merged
Merged
SITL: add ELRS simulator #26663
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Very, very cool! |
Implemented a buffers and rate limits within the simulator rather than using changing the UART driver, that makes it more representative and is less invasive in the UART driver. |
IamPete1
force-pushed
the
SIM_Radio
branch
2 times, most recently
from
April 2, 2024 02:15
a0bf577
to
f5f9be3
Compare
Rebased, squashed and moved from using the SITL UART driver to running a little TCP interface, this makes the diff much smaller. |
tridge
approved these changes
Apr 22, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This adds a simulator for a ELRS radio running the mavlink-rc branch. Nothing clever, just a data rate limit and some buffers with reporting via the RADIO_STATUS mavlink message.
It creates its own HAL SITL UART driver so all the usual commands for addresses and ports should still work. I have been testing with:You also have to set the serial baud to 460. Now runs its own little TCP server.-A --serial2=sim:ELRS:tcp:3
which allows you to connect tcp to 5763 (the same as normal for serial 2).The goal is to have a limited data link for testing both AP and GCS. EG (#26098)
A real ELRS setup:
This PR:
This also improves the UART data rate limiting by tracking the fractional number of bytes remainder on each call.