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

Jump to bottom

Expose home location as separate from .commands array #252

Closed
tcr3dr opened this issue Jul 28, 2015 · 3 comments
Closed

Expose home location as separate from .commands array #252

tcr3dr opened this issue Jul 28, 2015 · 3 comments
Assignees
@tcr3dr
Labels
enhancement
Milestone
Minor

Comments

@tcr3dr
Copy link
Contributor

tcr3dr commented Jul 28, 2015

#230 #207 #132

Home location doesn't act in the same way as our .commands array (see above topic for discussion). Instead of patching unexpected behavior, we can extract this "initial" command as a separate Vehicle.home_location variable.
@tcr3dr tcr3dr added this to the v1.5.0 milestone Jul 28, 2015
@tcr3dr tcr3dr self-assigned this Jul 28, 2015
@hamishwillee
Copy link
Contributor

The attribute should also be settable (the MAV_CMD_DO_SET_HOME command can be set any time)

FYI, the ardupilot implementation of home location as first waypoint is a "bug" that they hope to undo at some point.

@hamishwillee hamishwillee mentioned this issue Jul 29, 2015
Closed
@hamishwillee hamishwillee mentioned this issue Aug 9, 2015
Closed
@tcr3dr tcr3dr modified the milestones: Major, v1.5.0, Minor Aug 12, 2015
@hamishwillee hamishwillee mentioned this issue Sep 3, 2015
Open
@hamishwillee
Copy link
Contributor

This is fixed by #384

@tcr3dr
Copy link
Contributor Author

tcr3dr commented Nov 10, 2015

Complete!

@tcr3dr tcr3dr closed this as completed Nov 10, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@tcr3dr tcr3dr

Labels
enhancement
Projects
None yet
Milestone
Minor
Development

No branches or pull requests
2 participants
@hamishwillee @tcr3dr