PX4 Core Components & Release Workflows #3443
Conversation
* also introduces core concept list
mrpollo
force-pushed
the
pr_release_procedure
branch
from
1ca271e to
241a975
Compare
| - Maintainer | ||
| - Tester |
There was a problem hiding this comment.
That is desirable, but we haven't had a maintainer for MacOS forever, and many of these others too. So perhaps something less "must" ish
There was a problem hiding this comment.
Also, who is this tester?
| | Flightgear (Simulation) | | | | ||
| | JSBSim (Simulation) | | | | ||
| | jMAVSim (Simulation) | | | | ||
| | SIH (Simulation) | | | |
There was a problem hiding this comment.
The are all community maintained. I think it might be worth listing these separately. Only SIH has a maintainer, of sorts.
| | Flightgear (Simulation) | | | | |
| | JSBSim (Simulation) | | | | |
| | jMAVSim (Simulation) | | | | |
| | SIH (Simulation) | | | |
| | JSBSim (Simulation) | | | | ||
| | jMAVSim (Simulation) | | | | ||
| | SIH (Simulation) | | | | ||
| | Multivehicle Simulation | | | |
There was a problem hiding this comment.
This is a feature of a simulation type as I understand it, so would be a sub set of gazebo (say).
| | Multivehicle Simulation | | | | ||
| | [MAVLink (Communication)][MAVLink (Communication)] | | | | ||
|
|
||
| [uORB (Architecture)]: https://github.com/PX4/PX4-Autopilot/issues?q=label%3AUorb+ |
There was a problem hiding this comment.
Note, I pulled all the links out to make this table editable. This works better with automatic prettiering.
| | [Github Actions (Continuous Integration)][Github Actions (Continuous Integration)] | | | | ||
| | [Jenkins (Continuous Integration)][Jenkins (Continuous Integration)] | | | | ||
| | Linux (Toolchain) | | | | ||
| | Mac OS (Toolchain) | | | |
There was a problem hiding this comment.
@mrpollo Are you populating the list of maintainers here?
| | [Jenkins (Continuous Integration)][Jenkins (Continuous Integration)] | | | | ||
| | Linux (Toolchain) | | | | ||
| | Mac OS (Toolchain) | | | | ||
| | [Windows (Toolchain)][Windows (Toolchain)] | WSL2 | | |
There was a problem hiding this comment.
The windows toolchain is just WLS2 IMO. Cygwin and VMs are defunct or community supported. So I would remove the WSL2 subcomponent.
| | RTK GPS (Driver) | Heading estimate from dual RTK GPS | [Alex Klimaj](https://github.com/AlexKlimaj) | | ||
| | [Documentation][Documentation] | PX4, MAVLink, QGC, MAVSDK | [Hamish Willee](https://github.com/hamishwillee) | | ||
| | Battery (Library) | | | | ||
| | Flight Task (Library) | | | |
There was a problem hiding this comment.
| | Flight Task (Library) | | | | |
| | Flight Task (Library) | | [Matthias Grob](https://github.com/MaEtUgR) | |
| | [uORB (Architecture)][uORB (Architecture)] | | | | ||
| | [Parameter (Architecture)][Parameter (Architecture)] | | | | ||
| | Realtime Clock (Architecture) | | | | ||
| | Work queue (Architecture) | | | |
There was a problem hiding this comment.
IMO all of these are subcomponets of "architecture", owned by either/both Beat and Daniel.
| | Work queue (Architecture) | | | | ||
| | GNSS (State Estimation) | Velocity / Position | | | ||
| | | Heading | | | ||
| | [Optical flow (State Estimation)][Optical flow (State Estimation)] | | | |
There was a problem hiding this comment.
This is probably a subset of estimation, not its own thing.
|
No flaws found |
| @@ -9,3 +11,107 @@ A list of PX4 release notes, they contain a list of the changes that went into e | |||
| - [v1.12](../releases/1.12.md) | |||
|
|
|||
| The full archive of releases for the PX4 autopilot project can be found on [GitHub](https://github.com/PX4/PX4-Autopilot/releases) | |||
|
|
|||
There was a problem hiding this comment.
If the rest of the maintainers are happy with this, so am I.
One concern is that there are no time-frames for expectations. Now I know we can't say to a maintainer, you will have tested and fixed all issues by next week, but we can say something like "the intent is that we'll converge within 2 months to a candidate in which only critical issues will be fixed, and 3 months for release from branch-off. Maintainers and tested are therefore expected to report on their progress to these goals" (or similar). Something that ties accountability.
There was a problem hiding this comment.
OK, I've subedited and added some comments.
THe core components list needs another look - there are things like estimators etc that aren't mentioned that are pretty core. EKF!
UPshot, a bit broader at the component level, initially populated from your maintainer list.
Back to you @mrpollo
|
@mrpollo FYI, still on you. |
Splits up from #2331 by @junwoo091400