Skip to content

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

Open
r0n4n wants to merge 33 commits into
skybrush-io:mainfrom
r0n4n:rover_config
Open

Add a config to handle the display and the commands for the rovers#122
r0n4n wants to merge 33 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

volfpeter and others added 30 commits December 15, 2025 11:23
…arks-light-config

Feat: color support for sparks light config
# Conflicts:
#	src/components/dialogs/app-settings/UAVsTab.jsx
#	src/components/uavs/UAVOperationsButtonGroup.jsx
Note, that this commit also includes fixes for some issues raised by
`tseslint.configs.stylisticTypeChecked`, as it was enabled for a while
instead of `tseslint.configs.stylistic` while going through the list.
Switch to the double underscore convention to avoid having a parent
directory with no `index.d.ts`.
@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.

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.

4 participants