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

OSM downloader error #894

Closed
vanpuk opened this issue Mar 19, 2014 · 16 comments
Closed

OSM downloader error #894

vanpuk opened this issue Mar 19, 2014 · 16 comments
Milestone

Comments

@vanpuk
Copy link
Contributor

vanpuk commented Mar 19, 2014

Problem

Error Received on Mac - InaSAFE 2.0.4

Solution

Fix error???

screen shot 2014-03-19 at 8 12 27 am

@vanpuk
Copy link
Contributor Author

vanpuk commented Mar 19, 2014

I have swapped over to the master and did not sync so am working on 269abc9- 27Feb (very shameful I know) but its working, so I think the changes that have been made recently to the downloader has created this error

@akbargumbira
Copy link
Member

Kristy, that is related to Linfiniti server, I also got that error
sometimes. Are you sure it was working? because I tried to open the URL (
http://osm.linfiniti.com/buildings-shp?bbox=448204.622213,9816174.79464,539578.223809,9924009.59469&qgis_version=2&output_prefix=mentawai1)
via browser, I also got Internal Server Error

cc: @timlinux

On Wed, Mar 19, 2014 at 8:34 AM, Kristy Van Putten <notifications@github.com

wrote:

I have swapped over to the master and did not sync so am working on
269abc9 269abc9\- 27Feb (very
shameful I know) but its working, so I think the changes that have been
made recently to the downloader has created this error

Reply to this email directly or view it on GitHubhttps://github.com//issues/894#issuecomment-38009174
.

-------------------

Akbar Gumbira

Software Engineer - Risk and Vulnerability Team

Australia-Indonesia Facility for Disaster Reduction (AIFDR)

-----------------------------------------------------------------------------------------

Menara Thamrin, Jakarta E | akbargumbira@gmail.com

M | +62 856 24545876
W | www.aifdr.org

@timlinux
Copy link
Contributor

Hi

I am looking...

Regards

Tim

On Wed, Mar 19, 2014 at 4:54 AM, Akbar Gumbira notifications@github.comwrote:

Kristy, that is related to Linfiniti server, I also got that error
sometimes. Are you sure it was working? because I tried to open the URL (

http://osm.linfiniti.com/buildings-shp?bbox=448204.622213,9816174.79464,539578.223809,9924009.59469&qgis_version=2&output_prefix=mentawai1
)
via browser, and I got Internal Server Error

cc: @timlinux

On Wed, Mar 19, 2014 at 8:34 AM, Kristy Van Putten <
notifications@github.com

wrote:

I have swapped over to the master and did not sync so am working on
269abc9 269abc9\- 27Feb (very

shameful I know) but its working, so I think the changes that have been
made recently to the downloader has created this error

Reply to this email directly or view it on GitHub<
https://github.com/AIFDR/inasafe/issues/894#issuecomment-38009174>
.

-------------------

Akbar Gumbira

Software Engineer - Risk and Vulnerability Team

Australia-Indonesia Facility for Disaster Reduction (AIFDR)

-----------------------------------------------------------------------------------------

Menara Thamrin, Jakarta E | akbargumbira@gmail.com

M | +62 856 24545876
W | www.aifdr.org

Reply to this email directly or view it on GitHubhttps://github.com//issues/894#issuecomment-38012937
.

Tim Sutton

Visit http://linfiniti.com to find out about:

  • QGIS programming services
  • GeoDjango web development
  • QGIS Training
  • FOSS Consulting Services
    Skype: timlinux Irc: timlinux on #qgis at freenode.net
    Tim is a member of the QGIS Project Steering Committee

@timlinux
Copy link
Contributor

@akbargumbira it is the server giving an error, but its because we are passing in an invalid extents (it should be in geographics coordinates).

@vanpuk what CRS are you using and is on the fly projection working?

@akbargumbira
Copy link
Member

@timlinux: I never looked at osm.linfiniti.com to see how it handles the request. Should there be some validations regarding bbox parameter? And then we can inform the user from OSM downloader tools if that is the case

@timlinux
Copy link
Contributor

@akbargumbira That code is all behaving correctly - the problem is that the bbox is being passed in EPSG:3857 instead of geo coordinates.

@akbargumbira
Copy link
Member

@timlinux: yeah I see. I can replicate it (change the CRS to EPSG:3857) so, shall we handle it on OSM downloader tools itself? Just give user the warning that it should be in geo coordinates? or convert all kind of the projection to geo coordinates?

or handle it in osm reporter?

@timlinux
Copy link
Contributor

There is already logic for doing the conversion in place - its just not being called or it is returning the incorrect result. Leave this one to me, I will look at it, thanks!

timlinux added a commit to timlinux/inasafe that referenced this issue Mar 19, 2014
@timlinux
Copy link
Contributor

@vanpuk can you confirm which CRS you were using?

timlinux added a commit to timlinux/inasafe that referenced this issue Mar 19, 2014
…valid expected results and test_utilities has a green dot.
@timlinux
Copy link
Contributor

I've written some tests to replicate. So it seems this error is specific to QGIS 2.0. Can you confirm that you are using QGIS 2.0 vanpuk? A temporary work around would be to ensure your project is in EPSG:4326. When I find the pronlem I will hotfix and make a bugfix release.

timlinux added a commit to timlinux/inasafe that referenced this issue Mar 19, 2014
@timlinux
Copy link
Contributor

@vanpuk from my testing here it seems to replicate if I use EPSG:900913 in the unit tests, but if I switch to EPSG:3857 (the more 'official' google projection now) the problem does not occur.

So could you provide a little more detail:

  • Which CRS do you have enabled?
  • Which QGIS version do you have?
  • Do you have On the Fly reprojection enabled?

Thanks

timlinux added a commit to timlinux/inasafe that referenced this issue Mar 19, 2014
@akbargumbira
Copy link
Member

@vanpuk can you provide the details that Tim asked here? Thanks

@timlinux
Copy link
Contributor

I'll look at this for 2.2

@timlinux timlinux added this to the Version 2.2 milestone Oct 22, 2014
@timlinux timlinux added the Ready label Oct 22, 2014
@timlinux timlinux modified the milestones: Version 2.3, Version 2.2 Nov 27, 2014
@timlinux timlinux removed the Ready label Nov 27, 2014
@timlinux timlinux assigned Gustry and unassigned timlinux Mar 11, 2015
@timlinux
Copy link
Contributor

@Gustry Can you look into this and close if you can't replicate. Please test when your project is using projected CRS.

@Gustry
Copy link
Contributor

Gustry commented Mar 12, 2015

I can't replicate it.
With OTF enabled, it works.
With OTF disabled, I get
The bounding box is not valid. Please make sure it is valid or check your projection!
@timlinux Should the OSM downloader allow to download if OTF is disabled, and then activate it ?

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

No branches or pull requests

4 participants