-
Notifications
You must be signed in to change notification settings - Fork 14
Add option to start simulator via docker #944
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
Merged
Turnlings
merged 120 commits into
main
from
939-add-option-to-start-simulator-via-docker
Feb 22, 2026
Merged
Changes from all commits
Commits
Show all changes
120 commits
Select commit
Hold shift + click to select a range
98bcdd6
Add start simulation button
Turnlings 1d5983e
Fix port
Turnlings 4f84a19
Add stop simulation button
Turnlings c53ddfa
Working error messages (only if using string literal)
Turnlings 3294652
Move onSimulationResult to different set of events
Turnlings 2bdbc24
Add feedback of running state from backend
Turnlings 257e6e3
Give feedback on first time use
Turnlings 7e4fe81
Docker stubs
Turnlings 93380f2
Resolve minor feedback
Turnlings 37083db
Resolve more feedback
Turnlings b1625cb
Update requirements for stubs
Turnlings 1102d06
Exception handling for image pull
Turnlings 94087a7
Add docker to mypi ignore
Turnlings 671410d
Add simulation modal
Turnlings 3ce42cf
Add dropdown to select vehicle type
Turnlings 99f74d3
Add fields for the other ardupilot params
Turnlings 2919b67
Copy
Turnlings 5e2ed82
Pass params to simulator startup
Turnlings 1cbf25d
npx
Turnlings d54ed6e
Remove ArduRover for now as is not valid
Turnlings e26f8a1
Cleanup of args parsing
Turnlings 8a70694
Wait for YOU CAN CONNECT message
Turnlings c320748
Disable if not connected to socket
Turnlings 2d36aba
Add connect after simulator start
Turnlings 2da4927
linting
Turnlings bb9d008
Backend refactor and comments
Turnlings 8562cc8
Add port option to also allow for tests
Turnlings 05c69ec
Add cleanup to the first test
Turnlings d321343
Generate tests for the other functions
Turnlings 4afd189
Resolve minor copilot feedback
Turnlings 2cb1292
Lint
Turnlings e94774f
Pass connect arg through backend
Turnlings 20910eb
Use port 5763 for tests
Turnlings 05226dc
Potential fix for test
Turnlings 780b51c
Background task for wait for container result
Turnlings 944676b
Better error messages on stopping simulation
Turnlings a1543bb
CI friendly cleanup container
Turnlings 9a2d92f
Merge branch 'main' into 939-add-option-to-start-simulator-via-docker
Turnlings 2b9878e
Resolve minor copilot feedback
Turnlings 3af9c9a
Merge branch '939-add-option-to-start-simulator-via-docker' of https:…
Turnlings 18a5a78
Improve test coverage
Turnlings 7a12241
Remove container reload because it breaks logs
Turnlings 435c743
Potential test fix
Turnlings 362116d
Catch exceptions from streaming container
Turnlings 2fb81bb
Clamp input values
Turnlings 0808fbb
Image downloading progress messages
Turnlings 4b86a30
Message on closing modal
Turnlings 200835a
Errors and exceptions
Turnlings b79c390
More tests
Turnlings 9bd9c0f
Linting
Turnlings a5b2c0a
Remove unused import
Turnlings e32fe28
Update radio/app/endpoints/simulation.py
Turnlings cc035d8
Resolve final copilot comments
Turnlings c456267
Rename function in tests
Turnlings 95ef9b0
Resolve minor feedback
Turnlings da04b07
Fix non-deterministic tests
Turnlings a4faa78
minor copilot feedback
Turnlings b5cee21
Merge branch 'main' into 939-add-option-to-start-simulator-via-docker
Turnlings df1ecda
Close old notification if new loading has begun
Turnlings 569df21
Rewrite waiting for container
Turnlings 4b35112
Merge branch '939-add-option-to-start-simulator-via-docker' of https:…
Turnlings 5cff764
Resolve MORE minor copilot feedback
Turnlings 4cb7568
Remove extra parameters
Turnlings 7af0e1a
Cleanup missed references
Turnlings 76443c8
Use correct port on auto-connect
Turnlings 22c5ad8
Selection for both host and container ports
Turnlings f9ecf1b
Add more tests for failure routes
Turnlings 0a50fd0
Merge branch 'main' into 939-add-option-to-start-simulator-via-docker
Turnlings d57d3d4
Fix tests
Turnlings 1544fc5
Resolve copilot feedback
Turnlings 1b4d101
Just a couple more copilot comments
Turnlings 2db66e9
Minor copilot feedback
Turnlings 5c20d85
Even more copilot feedback
Turnlings 18072a9
Rewrite waiting function to use since and tail
Turnlings 12e13c1
Frontend changes
Turnlings 3293b6c
Get rid of all the tests
Turnlings 4cb4e33
Catch bubble up exception
Turnlings 4866e7e
Copy
Turnlings 92edfe2
Option for multiple port mappings
Turnlings 548692b
Refactor simulation params into their own slice
Turnlings addfe60
Merge branch 'main' into 939-add-option-to-start-simulator-via-docker
Turnlings e68fc06
Duplicate port validation
Turnlings 7db250f
Refactor port validation into helper functions
Turnlings f66e84b
Merge branch '939-add-option-to-start-simulator-via-docker' of https:…
Turnlings 3117a68
Fix None unpacking
Turnlings c467a12
Linting
Turnlings cddec7b
Use exceptions rather than returning None
Turnlings 978b12a
Store simulation loading id states
Turnlings cee715f
Linting
Turnlings ae381d2
Minor changes
Turnlings fbb2d14
Resolve what are becoming more and more pointless comments
Turnlings 707c33f
These edge cases are literally never gonna happen
Turnlings e67d9c0
Check for duplicate container ports aswell
Turnlings af8e295
Copy
Turnlings eda9ff0
Type annotations
Turnlings 91b89c2
Fix typing
Turnlings c7fff48
Fix the final none hallucinated issues
Turnlings f59e968
Linting
Turnlings 8372de2
Please
Turnlings f1900ec
Use uuids for ports
Turnlings d34877f
Use worker thread for log polling
Turnlings a9a8bcc
Remove unused reference to index
Turnlings 6ad46f0
MAYBE these ones are relevant
Turnlings 616d967
Merge branch 'main' into 939-add-option-to-start-simulator-via-docker
NexInfinite 6627adc
Hide button on default with opt-in experimental developer setting
Turnlings e0fc459
Resolve the trivial parts of the feedback
Turnlings 8d6665d
Use docker types
Turnlings f24d3ef
Add custom type for command parameters
Turnlings b73f422
Optional type, because it is
Turnlings 9ce80e6
Rename cleanup_container to stop_container
Turnlings 08acb65
Merge branch 'main' into 939-add-option-to-start-simulator-via-docker
Turnlings f0c1953
Use trash icon instead of X
Turnlings ecfc263
Add connection progress bar to simulation modal footer
Turnlings 5cc3018
Merge branch '939-add-option-to-start-simulator-via-docker' of https:…
Turnlings ae7b7c6
Remove now unused Progress import
Turnlings 3f537cf
Disconnect from drone when stopping simulation if connected to ANYTHING
Turnlings 25d64ca
Disconnect from simulator on stop simulation
Turnlings c72335e
Block connecting to simulator if already connected to drone
Turnlings 7631e0b
Merge branch 'main' into 939-add-option-to-start-simulator-via-docker
Turnlings ebc44be
Fix issue with merge
Turnlings File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or 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
This file contains hidden or 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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,22 @@ | ||
| import { Progress } from "@mantine/core" | ||
|
|
||
| export default function ConnectionProgress({ connecting, status }) { | ||
| return ( | ||
| <> | ||
| {connecting && | ||
| status.message !== null && | ||
| typeof status.progress === "number" && ( | ||
| <> | ||
| <p className="text-center my-4">{status.message}</p> | ||
| <Progress | ||
| animated | ||
| size="lg" | ||
| transitionDuration={300} | ||
| value={status.progress} | ||
| className="w-full mx-auto my-auto" | ||
| /> | ||
| </> | ||
| )} | ||
| </> | ||
| ) | ||
| } |
This file contains hidden or 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
This file contains hidden or 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
Oops, something went wrong.
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.
Uh oh!
There was an error while loading. Please reload this page.