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

Pass srid to postgis #36

Closed

Conversation

eddietejeda
Copy link
Contributor

No description provided.

@Mr0grog
Copy link
Member

Mr0grog commented Feb 10, 2013

@yuletide Can you take a look at this?

@yuletide
Copy link
Member

I'll look into this, might be due to different postgres versions

@eddietejeda
Copy link
Contributor Author

I did the research:

Version 1.0
Did not have ST_MakeEvelope

Version 1.5
http://postgis.net/docs/manual-1.5/ST_MakeEnvelope.html

Version 2.0
http://postgis.net/docs/manual-2.0/ST_MakeEnvelope.html

The difference between version 1.5 and version 2.0 is that in version 2.0 this sentence is added: If no SRID is specified the unknown spatial reference system is assumed.

@Mr0grog
Copy link
Member

Mr0grog commented Feb 13, 2013

Hmmm, if that were the case, then the old code should have worked fine in PostGIS 2.0. Isn't that the version where this was throwing errors on you, though, @eddietejeda? Seems odd.

@yuletide
Copy link
Member

+1

On Tuesday, February 12, 2013 at 5:28 PM, Rob Brackett wrote:

Hmmm, if that were the case, then the old code should have worked fine in PostGIS 2.0. Isn't that the version where this was throwing errors on you, though, @eddietejeda (https://github.com/eddietejeda)? Seems odd.


Reply to this email directly or view it on GitHub (#36 (comment)).

@eddietejeda
Copy link
Contributor Author

I am using PostGIS 2.0. I was using EPSG:4326 for my database projection. It might be that certain projection types are not automatically detected.

@Mr0grog
Copy link
Member

Mr0grog commented May 13, 2013

  1. This PR should probably be re-proposed in https://github.com/nodetiles/nodetiles-postgis, which is where the PostGIS source will live from now on.
  2. Can you send me or @yuletide a dump of the table you're using, @eddietejeda? Might be able to dig deeper into understanding exactly what's going wrong, what conditions this is needed for, etc.

@bensheldon bensheldon closed this Mar 30, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants