Skip to content

Docs: Overhaul rover documentation #25103

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
merged 4 commits into from
Jun 25, 2025
Merged

Docs: Overhaul rover documentation #25103

merged 4 commits into from
Jun 25, 2025

Conversation

chfriedrich98
Copy link
Contributor

Solved Problem

Recent rover development streamlined the different rover modules to a point that their parameters/setpoints are nearly identical.
This PR updates the rover docs to the latest changes and merges the configuration/tuning and flight mode documentation of the different rover modules into a single one.

@chfriedrich98 chfriedrich98 self-assigned this Jun 23, 2025
@chfriedrich98 chfriedrich98 added the Rover 🚙 Rovers and other UGV label Jun 23, 2025
@chfriedrich98 chfriedrich98 moved this to 🏗 In Progress in PX4 Rover Jun 23, 2025
@chfriedrich98 chfriedrich98 force-pushed the pr-rover_docs_update branch from cd3a76d to 6d8fbd4 Compare June 23, 2025 13:11
@chfriedrich98 chfriedrich98 marked this pull request as ready for review June 23, 2025 13:14
@chfriedrich98 chfriedrich98 moved this from 🏗 In Progress to 👀 In Review in PX4 Rover Jun 23, 2025
@@ -0,0 +1,44 @@
# Rover Configuration/Tuning
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do we need to point to other things people might need to do, such as setting up RC controller?

Comment on lines +35 to +39
```sh
CONFIG_MODULES_ROVER_ACKERMANN=y
CONFIG_MODULES_ROVER_DIFFERENTIAL=y
CONFIG_MODULES_ROVER_MECANUM=y
```
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Are these independent? I mean if I only want to use ackermann, would I only install that module?

@hamishwillee
Copy link
Contributor

Does this need a release note in main?

@hamishwillee hamishwillee force-pushed the pr-rover_docs_update branch from a8e018d to baf11f7 Compare June 25, 2025 02:36
Copy link

No flaws found

- The module currently supports [manual mode](../flight_modes_rover/differential.md#manual-mode), [acro mode](../flight_modes_rover/differential.md#acro-mode), [stabilized mode](../flight_modes_rover/differential.md#stabilized-mode), [position mode](../flight_modes_rover/differential.md#position-mode) and [auto modes](../flight_modes_rover/differential.md#auto-modes).
- New module dedicated to [mecanum rovers](../frames_rover/mecanum.md):
- The module currently supports [manual mode](../flight_modes_rover/mecanum.md#manual-mode), [acro mode](../flight_modes_rover/mecanum.md#acro-mode), [stabilized mode](../flight_modes_rover/mecanum.md#stabilized-mode), [position mode](../flight_modes_rover/mecanum.md#position-mode) and [auto modes](../flight_modes_rover/mecanum.md#auto-modes).
- New firmware build specifically for [rovers](../config_rover/index.md#flashing-the-rover-build).
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

FYI only, I'm hoping that we are able to manage docs builds properly for v1.16 and that these updates won't leak into that docs build. I'm not all that hopeful.

@hamishwillee
Copy link
Contributor

hamishwillee commented Jun 25, 2025

@chfriedrich98 Looks great - not just the docs, but the whole architecture feels more clean.

I've subedited, and updated the parameter docs so those broken links are gone. I'm going to merge this, but note the comments - we might need a release note for this.

@hamishwillee hamishwillee merged commit 81ec385 into main Jun 25, 2025
6 of 8 checks passed
@hamishwillee hamishwillee deleted the pr-rover_docs_update branch June 25, 2025 03:03
@github-project-automation github-project-automation bot moved this from 👀 In Review to ✅ Done in PX4 Rover Jun 25, 2025
@hamishwillee hamishwillee restored the pr-rover_docs_update branch June 25, 2025 03:03
@chfriedrich98 chfriedrich98 deleted the pr-rover_docs_update branch June 30, 2025 07:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Rover 🚙 Rovers and other UGV
Projects
Status: ✅ Done
Development

Successfully merging this pull request may close these issues.

2 participants