Skip to content

Add a config to handle the display and the commands for the rovers#122

Closed
r0n4n wants to merge 78 commits into
skybrush-io:mainfrom
r0n4n:rover_config
Closed

Add a config to handle the display and the commands for the rovers#122
r0n4n wants to merge 78 commits into
skybrush-io:mainfrom
r0n4n:rover_config

Conversation

@r0n4n
Copy link
Copy Markdown

@r0n4n r0n4n commented Jan 28, 2026

Config added to select the vehicle type:
image

Rover commands added for the rover configuration
image

Linked Skybrush server PR : skybrush-io/skybrush-server#13
Linked flockwave-spec : skybrush-io/flockwave-spec#4

r0n4n added 3 commits January 3, 2026 20:33
# Conflicts:
#	src/components/dialogs/app-settings/UAVsTab.jsx
#	src/components/uavs/UAVOperationsButtonGroup.jsx
@r0n4n
Copy link
Copy Markdown
Author

r0n4n commented Jan 28, 2026

I still need to fix the merge conflicts

r0n4n added 2 commits February 1, 2026 23:06
# Conflicts:
#	src/components/dialogs/app-settings/UAVsTab.jsx
#	src/components/uavs/UAVOperationsButtonGroup.jsx
…into rover_config

# Conflicts:
#	src/components/dialogs/app-settings/UAVsTab.jsx
@r0n4n
Copy link
Copy Markdown
Author

r0n4n commented Apr 4, 2026

@ntamas do you have any comment on my changes ?

@ntamas
Copy link
Copy Markdown
Member

ntamas commented Apr 5, 2026

Can you please change the target branch of the PR to dev instead of main? I see that you merged changes from dev into your branch but the PR is targeted at main so there are lots of formatting and linter changes that are unrelated to the subject of the PR, making things harder to review.

I tried to change the target branch myself but apparently Github does not allow it if I am not the owner of the branch.

ntamas and others added 21 commits April 22, 2026 23:10
zarcell and others added 27 commits April 30, 2026 11:04
…h-io/live into feat/display-num-drones-selected
Ref: https://github.com/openlayers/openlayers/releases/tag/v10.9.0
"In the 10.9 release [...] GeoTIFF support saw major improvements"
The style was broken in `a1ddc548` (MUI v5 upgrade) and the switch event
handling was incorrect since `0ee5d408` (refactoring to TypeScript)
Currently translated at 71.9% (708 of 984 strings)

Translation: Skybrush/Live
Translate-URL: http://translate.skybrush.io/projects/skybrush/live/zh_Hans/
Currently translated at 100.0% (984 of 984 strings)

Translation: Skybrush/Live
Translate-URL: http://translate.skybrush.io/projects/skybrush/live/zh_Hans/
Currently translated at 100.0% (984 of 984 strings)

Translation: Skybrush/Live
Translate-URL: http://translate.skybrush.io/projects/skybrush/live/en/
…utton does not respond to click events when disabled
@CLAassistant
Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
4 out of 6 committers have signed the CLA.

✅ volfpeter
✅ zarcell
✅ vasarhelyi
✅ r0n4n
❌ Copilot
❌ weblate
You have signed the CLA already but the status is still pending? Let us recheck it.

@r0n4n
Copy link
Copy Markdown
Author

r0n4n commented May 17, 2026

New pull request created to merge the new feature to the dev branch

@r0n4n r0n4n closed this May 17, 2026
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.

8 participants