-
Notifications
You must be signed in to change notification settings - Fork 13.6k
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
Reposition vs Mission Pause #12544
Comments
Notes from dev call:
|
This was recently brought up in slack again and there was the idea to use This is a related issue for goto/reposition executing in its own mode: #14576 |
@MaEtUgR @DonLakeFlyer et al Using the existing pause message would be much more desired. Why don't we do this? Regarding MAV_CMD_DO_REPOSITION hacks, I'm not opposed to this as well as long as the documentation for the message in common.xml gets updated and everyone adopts the same approach. Is this what is expected to happen? |
Do you mean this: MAV_CMD_DO_PAUSE_CONTINUE. First I've seen that. Does it actually work? |
Probably not not on PX4, or this issue would be closed. Looks like we do respond to it in PX4 v1.11: 509be82 But implementing this is the "right thing to do" from a mavlink perspective (ie this is the command that meets the intention) |
This issue has been automatically marked as stale because it has not had recent activity. Thank you for your contributions. |
To my understanding for both
GoTo
and forMission-Pause
the mavlink-msg MAV_CMD_DO_REPOSITION(192) is used. However, there is already a dedicated msg for pausing/continuing a mission called MAV_CMD_DO_PAUSE_CONTINUE (193 ). To be able to distinguish betweenGoTo
andPause
also within the ulog, it would be beneficial to use 193 for mission-pause instead of 192.Given that such a change requires changes on both PX4 and QGC, I opened the same issue on QGC as well (mavlink/qgroundcontrol#7612).
The text was updated successfully, but these errors were encountered: