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

Enhance DistanceUnit to recognize nautical miles. #5085

Closed
brian-from-fl opened this issue Feb 11, 2014 · 1 comment
Closed

Enhance DistanceUnit to recognize nautical miles. #5085

brian-from-fl opened this issue Feb 11, 2014 · 1 comment
Labels
:Analytics/Geo Indexing, search aggregations of geo points and shapes >enhancement v1.1.0 v2.0.0-beta1

Comments

@brian-from-fl
Copy link

The DistanceUnit class should add the DistanceUnit.NAUTICALMILES enumeration. For example:

NAUTICALMILES(1852.0, "NM", "nmi"),

I'll let someone else decide what the second, longer, string should actually be. Most aeronautical charts use "nm" but I've also seen "nmi".
@brian-from-fl brian-from-fl reopened this Feb 11, 2014
@chilling chilling self-assigned this Feb 11, 2014
chilling pushed a commit that referenced this issue Feb 14, 2014
@chilling
Added the DistanceUnit.NAUTICALMILES enumeration
d0ce772 
label with the corresponding *NM* and *nmi* unit
suffixes. Update the docs to match.

Closes #5085
chilling pushed a commit that referenced this issue Feb 14, 2014
@chilling
Added the DistanceUnit.NAUTICALMILES enumeration
a154b3c 
label with the corresponding *NM* and *nmi* unit
suffixes. Update the docs to match.

Closes #5085
@chilling chilling added v1.1.0 and removed v1.0.1 labels Feb 14, 2014
@chilling
Copy link
Contributor

@brian-from-fl thanks a lot

@clintongormley clintongormley added the :Analytics/Geo Indexing, search aggregations of geo points and shapes label Jun 7, 2015
@brian-from-fl brian-from-fl mentioned this issue Jun 7, 2015
Closed
mute pushed a commit to mute/elasticsearch that referenced this issue Jul 29, 2015
@chilling
Added the DistanceUnit.NAUTICALMILES enumeration
14c543c 
label with the corresponding *NM* and *nmi* unit
suffixes. Update the docs to match.

Closes elastic#5085
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
:Analytics/Geo Indexing, search aggregations of geo points and shapes >enhancement v1.1.0 v2.0.0-beta1
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add support for distances in nautical miles brian-from-fl/elasticsearch
3 participants
@clintongormley @chilling @brian-from-fl