bjornharrtell / jts2geojson

JTS from/to GeoJSON converter for Java
MIT License
141 stars 57 forks source link

CRS support #7

Closed mplatt closed 8 years ago

mplatt commented 8 years ago

Is there any support for named CRS planned? Specifically, I would love to use something along the lines of org.geolatte.common.dataformats.json.jackson.GeoJsonDeserializer to translate the CRS into an SRID for jts. Do you feel this kind of functionality would make any sense in this package?

bjornharrtell commented 8 years ago

Makes sense. No plans. :)

mplatt commented 8 years ago

Cool stuff, thanks for the info. I might prepare a PR then

ngocketit commented 3 years ago

Hi. What's the status of this @mplatt ?

bjornharrtell commented 3 years ago

I would say it is safe to assume interest wasn't that great after all as it was more than 5 years ago this was closed with no follow up. ๐Ÿ˜‰

mplatt commented 3 years ago

Yea, sorry guys. I have to admit I don't even understand my own request anymore ๐Ÿ˜ƒ

On Mon, 25 Jan 2021, 21:02 Bjรถrn Harrtell, notifications@github.com wrote:

I would say it is safe to assume interest wasn't that great after all as it was more than 5 years ago this was closed with no follow up. ๐Ÿ˜‰

โ€” You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/bjornharrtell/jts2geojson/issues/7#issuecomment-767110409, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAMWDGWY3HWIVSGRXZKDV7TS3XL7ZANCNFSM4BXFQSSA .

ngocketit commented 3 years ago

@bjornharrtell Yes I sort of agree since it's not a hard requirement for me to have named CRS supported. Basically what I did is to assume the geometry that comes from the geojson is with the targeted SRID and use ST_SetSrid after ST_GeomFromGeoJSON. With CRS in geojson, I can omit ST_SetSrid.