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

Allow different co-ordinate systems for geometric objects. #3590

Closed
keedon opened this issue Aug 28, 2013 · 8 comments
Closed

Allow different co-ordinate systems for geometric objects. #3590

keedon opened this issue Aug 28, 2013 · 8 comments
Assignees
Labels
:Analytics/Geo Indexing, search aggregations of geo points and shapes >docs General docs changes good first issue low hanging fruit help wanted adoptme

Comments

@keedon
Copy link

keedon commented Aug 28, 2013

It looks like elasticsearch only allows WGS-84 coordinates for geometric objects, the application I'm currently working on uses OSGB-36 which throws an exception when you try to insert a node with co-ordinates outside of the allowed range for WGS-84

Thanks

@chilling
Copy link
Contributor

chilling commented Sep 6, 2013

Hi @keedon, you're right. ES only supports WGS84 coordinates for geodata. I'm thinking about extending the mappings by a projection definition, but this may take a while.

Thanks for coming up with this great idea,
Florian

@ghost ghost assigned chilling Sep 6, 2013
@yeroc
Copy link

yeroc commented Sep 11, 2013

Curious. We just happened to run into this limitation ourselves. Could the documentation be updated to at least be more explicit in this regard (until this enhancement is resolved)? The stacktrace that's generated really isn't clear at all. We had to do some digging in the source code to realize that projected coordinates and the GeoJson crs parameter was being completely ignored...

@colings86
Copy link
Contributor

We don't currently plan to support multiple co-ordinate systems within Elasticsearch. There are a lot of different coordinate systems and it would be difficult to reliably provide support for them all. I would suggest that you do the conversion from your co-ordinate system to WGS-84 in your client application.

We do need to update the documentation to be clear that we use WGS-84.

@colings86 colings86 added adoptme and removed discuss labels Sep 5, 2014
@clintongormley clintongormley added discuss :Analytics/Geo Indexing, search aggregations of geo points and shapes >docs General docs changes help wanted adoptme and removed help wanted adoptme >bug good first issue low hanging fruit >docs General docs changes discuss labels Sep 21, 2015
@clintongormley clintongormley added the good first issue low hanging fruit label Oct 14, 2015
@palecur palecur self-assigned this Dec 2, 2015
@desruisseaux
Copy link

Just in case the information may be of some use: Apache SIS 0.6 can create Mercator, Transverse Mercator, Lambert Conical and Stereographic map projections for a Coordinate Reference System defined in Well Known Text (WKT) format. It supports both version 1 and 2 of WKT. Apache SIS 0.7 (to be released in one or two months) will add support for EPSG codes, datum shifts and a few more map projections.

@clintongormley clintongormley assigned debadair and unassigned palecur Jan 15, 2016
@clintongormley
Copy link
Contributor

@nknize would you consider adding support for different coordinate systems?

@desruisseaux
Copy link

If there is any interest to start a thread or a wiki page on this subject, I may be able to help with some proposals on the coordinate transformations part.

@zmays
Copy link

zmays commented Feb 9, 2017

Please update the documentation - if one sends anything but wgs84, for example, web mercator, like this:
{"location":{"type":"GeometryCollection","crs" : { "type" : "name", "properties" : { "name" : "urn:ogc:def:crs:EPSG::3857" } },"geometries":[{"type":"MultiPolygon","coordinates":[ [ [ [ -11458202.599282084, 3936157.3620158276 ], [ -11459165.477255989, 3936155.366018667 ], [ -11459163.694921646, 3935177.6040588915 ], [ -11458202.66477565, 3935177.7214682177 ], [ -11458202.599282084, 3936157.3620158276 ] ] ], [ [ [ -11458202.599282084, 3936157.3620158276 ], [ -11457239.720985424, 3936159.277695415 ], [ -11457237.807021441, 3937140.8780615395 ], [ -11458202.533897068, 3937137.0839141724 ], [ -11458202.599282084, 3936157.3620158276 ] ] ] ] }]}}
this is the message one gets:
{ "error": { "root_cause": [ { "type": "mapper_parsing_exception", "reason": "failed to parse [location]" } ], "type": "mapper_parsing_exception", "reason": "failed to parse [location]", "caused_by": { "type": "null_pointer_exception", "reason": null } }, "status": 400 }

Which takes a long time to figure out what is going on.

Enhaloed added a commit to Enhaloed/elasticsearch that referenced this issue Mar 24, 2017
kblincoe added a commit to kblincoe/elasticsearch that referenced this issue Mar 24, 2017
kblincoe added a commit to kblincoe/elasticsearch that referenced this issue Apr 4, 2017
elastic#3590 Documentation now states coordinates format
Sue-Gallagher added a commit to Sue-Gallagher/elasticsearch that referenced this issue Mar 29, 2018
@Sue-Gallagher
Copy link
Contributor

Closed by #29305

jasontedor added a commit to jasontedor/elasticsearch that referenced this issue Apr 3, 2018
* master: (80 commits)
  Remove HTTP max content length leniency (elastic#29337)
  Begin moving XContent to a separate lib/artifact (elastic#29300)
  Java versions for ci (elastic#29320)
  Minor cleanup in the InternalEngine (elastic#29241)
  Clarify expectations of false positives/negatives (elastic#27964)
  Update docs on vertex ordering (elastic#27963)
  Revert "REST high-level client: add support for Indices Update Settings API (elastic#28892)" (elastic#29323)
  [test] remove Streamable serde assertions (elastic#29307)
  Improve query string docs (elastic#28882)
  fix query string example for boolean query (elastic#28881)
  Resolve unchecked cast warnings introduced with elastic#28892
  REST high-level client: add support for Indices Update Settings API (elastic#28892)
  Search: Validate script query is run with a single script (elastic#29304)
  [DOCS] Added info on WGS-84. Closes issue elastic#3590 (elastic#29305)
  Increase timeout on Netty client latch for tests
  Build: Use branch specific refspec sysprop for bwc builds (elastic#29299)
  TEST: trim unsafe commits before opening engine
  Move trimming unsafe commits from engine ctor to store (elastic#29260)
  Fix incorrect geohash for lat 90, lon 180 (elastic#29256)
  Do not load global state when deleting a snapshot (elastic#29278)
  ...
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 >docs General docs changes good first issue low hanging fruit help wanted adoptme
Projects
None yet
Development

No branches or pull requests

10 participants