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

Survey (Grid) Imperial Support, + random clean-up. Issue: #541 #565

Merged
merged 1 commit into from Aug 11, 2014

Conversation

tristannovak
Copy link

@meee1 I think I got it(git usage) right this time.

Still have to set altitude in meters, although it will add WP's in feet.
Issue: #541 addressed.

Still have to set altitude in meters, although it will add WP's in feet.
@tristannovak tristannovak changed the title Survey (Grid) Imperial Support, + random clean-up. Survey (Grid) Imperial Support, + random clean-up. Issue: #541 Aug 11, 2014
@tristannovak
Copy link
Author

imperial

meee1 added a commit that referenced this pull request Aug 11, 2014
Survey (Grid) Imperial Support, + random clean-up. Issue: #541
@meee1 meee1 merged commit 54aa2bd into ArduPilot:master Aug 11, 2014
@iskess
Copy link

iskess commented Aug 19, 2014

Tristan, I use imperial but after using the latest beta I am finding the mix of units confusing. Altitude is still in meters, while resolution is in cm. Most people who use Imperial do so because in aviation the international ICAO standard is in feet (eg 400ft limit). On the other hand, ground resolution is almost always referred to in cm. Here are some possible solutions:

  1. Always display both units, at least for key values like I named above, or
  2. Include a toggle switch inside the Survey tool so we can switch back and forth quickly as needed.
    I'm usually adjusting my grid to get to a target resolution in cm/pixel.

@iskess
Copy link

iskess commented Aug 27, 2014

@tristannovak,
Also, in Imperial mode, the "Distance between images" shows as feet, but the CAM_TRIGG_DIST parameter requires meters. It would really be helpful to have an instant toggle switch in the lower "stats" pane to display either type of unit.
Is there a way I can contact you more directly? Perhaps a username for DIYDrones of RCGroups so I can PM you? I'm iskess everywhere including hotmail.com

@bbasso
Copy link

bbasso commented Sep 2, 2014

@tristannovak Nice work here! I agree with @iskess - I was just unconfident enough in what units where what that I just did everything in meters. The altitude is the biggest issue in my mind.

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

Successfully merging this pull request may close these issues.

None yet

4 participants