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

Auto-insert RTL at end of 3D mapping mission #953

Closed
zlite opened this issue Jul 30, 2014 · 8 comments
Closed

Auto-insert RTL at end of 3D mapping mission #953

zlite opened this issue Jul 30, 2014 · 8 comments

Comments

@zlite
Copy link
Contributor

zlite commented Jul 30, 2014

If a circle mode 3D mapping mission is invoked, automatically add a RTL-equivalent waypoint after that returns to home and lands. (Right now I have to add that manually each time, which is a hassle)

@squilter
Copy link
Member

@arthurbenemann would it be appropriate to put this in sendMissionToAPM() in org.droidplanner.core.mission.Mission ?

A quick check to make sure no land or RTL is already in missionItems, then append an RTL.

@Thalek
Copy link

Thalek commented Jul 31, 2014

My only comment is that the moment someone comes up with a mission that doesn't want an RTL at the end, this feature will become a bug.

@arthurbenemann
Copy link
Member

Let's make the check on the upload event, and show a dialog to the user querying about start and finish of mission.

@chriselsen
Copy link

How about creating a new section under settings, called "Mission Planning".
Then under this section add the configuration item "Always complete Mission with RTL", which is enabled by default.
This way Chris gets his desired behavior, Thalek's concern is addressed and users aren't bothered by a million popups asking "Do you really want to...".

@Thalek
Copy link

Thalek commented Sep 26, 2014

Works for me. What's your opinion, Arthur?

@arthurbenemann
Copy link
Member

@Thalek I'm ready to merge the PR if someone writes it.

@Thalek
Copy link

Thalek commented Sep 26, 2014

Makes me wish my coding skills weren't so out of date. [wry smile]

@arthurbenemann
Copy link
Member

Done at #1142

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

No branches or pull requests

5 participants