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.

Sign up for GitHub

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

Jump to bottom

Feature request: Add option to reset the home position only the first time the craft is armed #3143

Closed
Dr-Drone opened this issue May 1, 2018 · 6 comments
Closed

Feature request: Add option to reset the home position only the first time the craft is armed #3143

Dr-Drone opened this issue May 1, 2018 · 6 comments
Milestone
2.0

Comments

@Dr-Drone
Copy link

Dr-Drone commented May 1, 2018

I was thinking if it was possible to not have the home point reset when disarmed and have the home point saved only when powered on. This may help save the aircraft if at all it decides or we accidentally disarm in flight.

Another feature would be to have iNAV return to home maintaining the current altitude and once it reaches home it could then lower it's altitude.
@stronnag
Copy link
Collaborator

stronnag commented May 1, 2018

I'm sure we discussed this before (can't recall where) and came to the conclusion that the best solution would a setting along the lines of:

set  inav_reset_home =  FIRST_ARM
Allowed values: NEVER, FIRST_ARM, EACH_ARM
# just like inav_reset_altitude

Or maybe one inav_reset_important_stuff setting that does both.

@Dr-Drone
Copy link
Author

Dr-Drone commented May 2, 2018

Yeah that seems just about right.

@danarrib
Copy link
Contributor

danarrib commented May 7, 2018

This is a VERY important feature. Anyone's planning to do it?

fiam added a commit that referenced this issue May 7, 2018
@fiam
Add new parameter for controlling home position reset
954ae18 
Add new inav_reset_home setting with the same possible values and
semantics as inav_reset_altitude (as suggested by @stronagg),
but with a default value of EACH_ARM to keep backwards compatibility.

Fixes #3143
@fiam fiam mentioned this issue May 7, 2018
Merged
@fiam fiam added this to the 2.0 milestone May 7, 2018
@fiam fiam changed the title Feature request: Mid Flight Disarm Feature request: Add option to reset the home position only the first time the craft is armed May 7, 2018
@danarrib
Copy link
Contributor

danarrib commented May 8, 2018

Awesome! Looking forward to test it!

@Dr-Drone
Copy link
Author

Dr-Drone commented May 8, 2018

Nice! 👏

@fiam
Copy link
Member

fiam commented Jun 7, 2018

Fixed in #3143

@fiam fiam closed this as completed Jun 7, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
2.0
Development

No branches or pull requests
4 participants
@fiam @stronnag @danarrib @Dr-Drone