search

cridenour / go-postgis

PostGIS support for Go. Works with any Postgres driver.
MIT License
108 stars 26 forks source link

Linestring support? #4

Open cherrymud opened 7 years ago

cherrymud commented 7 years ago

Is linestring currently supported? If so, would you provide an example? (I saw you removed some linestring code recently - I'm not sure if it was replaced by something else?)

If linestring isn't currently supported, any idea when you'll add it back in?

cridenour commented 7 years ago

Hey @cherrymud - sorry the delay. Linestring was part of a PR that ended up causing more problems than it solved. I'll take a look next week to see if I can come up with a simple implementation.

anthonybishopric commented 1 year ago

hey @cridenour ! I'd be happy to contribute linestring support, could you say more about what the reverted PR did? I know it's been a minute, let me know if you can provide guidance

cridenour commented 1 year ago

Hi @anthonybishopric - it was reverted after #3 was reported. I think they ended up changing too much about the processing of the library where I think a simple additional structure and linestring type ID could work? It's hard to say, it's been so many years :joy:

Sad to say I've always been more optimistic about investing more time into this library but definitely open to more PR requests (especially the code in #7 if others can test it).

Thanks!