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

Adding MicroStrain driver documentation #3426

Open
wants to merge 8 commits into
base: main
Choose a base branch
from

Conversation

JoelJ18
Copy link

@JoelJ18 JoelJ18 commented Oct 31, 2024

Documentation for MicroStrain Inertial Sensors
Related pull request:

@hamishwillee
Copy link
Collaborator

Thanks. For some reason github will let me commit, but not push the changed doc name etc up to your repo. So you'll have to rename the file.

@hamishwillee
Copy link
Collaborator

hamishwillee commented Nov 13, 2024

Thanks, this can merge after the associated PR. Note that after that merges the docs system will populate the missing parameter references in #3426 (comment), so we can rebase and the build errors will go away.

Ping me if you merge the other PR and I don't merge this shortly afterwards (I don't work every day, but I can certainly push this in once that happens)

Copy link

/en/sensor/microstrain.md

  • LinkedFileMissingAnchor: #MS_IMU_RATE_HZ not found in ../advanced_config/parameter_reference.md (/home/runner/work/PX4-user_guide/PX4-user_guide/en/advanced_config/parameter_reference.md)
  • LinkedFileMissingAnchor: #MS_MAG_RATE_HZ not found in ../advanced_config/parameter_reference.md (/home/runner/work/PX4-user_guide/PX4-user_guide/en/advanced_config/parameter_reference.md)
  • LinkedFileMissingAnchor: #MS_BARO_RATE_HZ not found in ../advanced_config/parameter_reference.md (/home/runner/work/PX4-user_guide/PX4-user_guide/en/advanced_config/parameter_reference.md)

@@ -9,6 +9,7 @@ The driver currently supports the following hardware:

- [`MicroStrain CV7-AHRS`](https://www.microstrain.com/inertial-sensors/3dm-cv7-ahrs): Inertial Measurement Unit (IMU) and Attitude Heading Reference System (AHRS)
- [`MicroStrain CV7-AR`](https://www.microstrain.com/inertial-sensors/3dmcv7-ar): Inertial Measurement Unit (IMU) and Vertical Reference Unit (VRU)
- [`MicroStrain CV7-INS`](https://www.microstrain.com/inertial-sensors/3dmcv7-ins): Inertial Measurement Unit (IMU) and Inertial Navigation System (INS). Only sensor output (gyro, accel, mag) is currently available. Support for the full navigation solution output will be added in a future driver release.
Copy link
Collaborator

Choose a reason for hiding this comment

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

We use sentence line breaks. Not a big deal but.

Suggested change
- [`MicroStrain CV7-INS`](https://www.microstrain.com/inertial-sensors/3dmcv7-ins): Inertial Measurement Unit (IMU) and Inertial Navigation System (INS). Only sensor output (gyro, accel, mag) is currently available. Support for the full navigation solution output will be added in a future driver release.
- [`MicroStrain CV7-INS`](https://www.microstrain.com/inertial-sensors/3dmcv7-ins): Inertial Measurement Unit (IMU) and Inertial Navigation System (INS).
Only sensor output (gyro, accel, mag) is currently available.
Support for the full navigation solution output will be added in a future driver release.

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.

2 participants