Skip to content
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

Create 5002_rover_r1 #21700

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Create 5002_rover_r1 #21700

wants to merge 1 commit into from

Conversation

NatchanonPai
Copy link

Solved Problem

When ... I found that ...

Fixes #{Github issue ID}

Solution

  • Add ... for ...
  • Refactor ...

Changelog Entry

For release notes:

Feature/Bugfix XYZ
New parameter: XYZ_Z
Documentation: Need to clarify page ... / done, read docs.px4.io/...

Alternatives

We could also ...

Test coverage

Context

Related links, screenshot before/after, video

@junwoo091400
Copy link
Contributor

Hi, do you have test logs with the real vehicle using this airframe file?

@MaEtUgR
Copy link
Member

MaEtUgR commented Dec 14, 2023

@NatchanonPai Do you use original Roboclaw motor driver? Did you consider hooking it up over UART/Serial? Then you can used the revised Roboclaw driver which supports wheel telemetry: #22215
With the revision came an updated R1 airframe configuration which should work out of the box:

param set-default CA_AIRFRAME 6
param set-default CA_R_REV 3
param set-default RBCLW_ADDRESS 128
param set-default RBCLW_FUNC1 101
param set-default RBCLW_FUNC2 102
param set-default RBCLW_REV 1

Here's the documentation for it in progress: PX4/PX4-user_guide@main...roboclaw-docs

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants