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

Geo: improve handling of out of bounds points in linestrings #47939

Merged
merged 4 commits into from
Oct 23, 2019

Conversation

imotov
Copy link
Contributor

@imotov imotov commented Oct 11, 2019

Brings handling of out of bounds points in linestrings in line with
points. Now points with latitude above 90 and below -90 are handled
the same way as for points by adjusting the longitude by moving it by
180 degrees.

Relates to #43916

Brings handling of out of bounds points in linestrings in line with
points. Now points with latitude above 90 and below -90 are handled
the same way as for points by adjusting the longitude by moving it by
180 degrees.

Relates to elastic#43916
@imotov imotov added >bug :Analytics/Geo Indexing, search aggregations of geo points and shapes v8.0.0 v7.5.0 labels Oct 11, 2019
@imotov imotov requested review from talevy and iverase October 11, 2019 17:21
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-analytics-geo (:Analytics/Geo)

Copy link
Contributor

@talevy talevy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

left some questions!

@imotov imotov added v7.6.0 and removed v7.5.0 labels Oct 22, 2019
Copy link
Contributor

@talevy talevy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks!

@imotov
Copy link
Contributor Author

imotov commented Oct 22, 2019

@elasticmachine run elasticsearch-ci/default-distro

@imotov
Copy link
Contributor Author

imotov commented Oct 22, 2019

@elasticmachine run elasticsearch-ci/packaging-sample-matrix

1 similar comment
@imotov
Copy link
Contributor Author

imotov commented Oct 23, 2019

@elasticmachine run elasticsearch-ci/packaging-sample-matrix

@imotov imotov merged commit 2e7d62c into elastic:master Oct 23, 2019
imotov added a commit that referenced this pull request Oct 23, 2019
Brings handling of out of bounds points in linestrings in line with
points. Now points with latitude above 90 and below -90 are handled
the same way as for points by adjusting the longitude by moving it by
180 degrees.

Relates to #43916
@imotov imotov deleted the issue-43916-out-of-bounds-linestrings branch May 1, 2020 22:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Analytics/Geo Indexing, search aggregations of geo points and shapes >bug v7.6.0 v8.0.0-alpha1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants