-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
add page for PX4 ROS library #2740
Changes from 1 commit
e49b3e5
ade0c99
4ee6bec
ae41744
997eaf9
58776d4
c8329b0
a18245d
dd58c6d
f8f7963
f0e7771
9e0a1f1
eff7f79
8205a4e
a3993ab
dc469fa
45f3842
592e5b9
d599ad7
fd5bde0
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,5 +1,17 @@ | ||
# PX4 ROS 2 Interface Library | ||
|
||
:::warning Experimental | ||
At time of writing parts of the PX4 ROS 2 Interface Library is experimental, and hence subject to change: | ||
|
||
- The architecture and core interfaces for defining modes in ROS 2 modes are largely stable. | ||
The library offers significant benefits over using offboard mode in its current state. | ||
- Only a few setpoint types have settled (the others are still under development). | ||
You may need to use internal PX4 topics which may not remain backwards-compatible over time. | ||
- The API is not fully documented. | ||
- Testing in CI is still limited. | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This is ok, but just fyi most of the things that are there are tested. Mostly the setpoint types (with their expected behavior) that are not. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Then let's say that :-)
hamishwillee marked this conversation as resolved.
Show resolved
Hide resolved
|
||
|
||
::: | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. FYI I added section "Experimental" up the top. What I'm trying to do here is explain that the mode stuff works now and already gives you a benefit over using offboard mode, but that many of the abstractions we provide for working with ROS 2 do not exist yet. So you can use it but likely things will change. I've done it in bullets so you can extend easily if you want. I'm open to any text which sets the level of expectation appropriately (i.e. this is "for discussion"). |
||
|
||
The [PX4 ROS 2 Interface Library](https://github.com/Auterion/px4-ros2-interface-lib) is a C++ library that simplifies controlling PX4 from ROS 2. | ||
|
||
Developers use the library to create and dynamically register modes written using ROS 2. | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
FYI the important thing here is to set expectation. So I have put a prefix experimental and added a link to the initial PR.
As you enhance this you can add new PR links.
If you consider significant parts of this stable, we can remove "Experimental".