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.

Sign up for GitHub

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

Jump to bottom

gdaltools relies on gdalinfo for raster SRS and extent, should use python API #18052

Closed
qgib opened this issue Jan 30, 2014 · 3 comments
Closed

gdaltools relies on gdalinfo for raster SRS and extent, should use python API #18052

qgib opened this issue Jan 30, 2014 · 3 comments
Labels
Bug Either a bug report, or a bug fix. Let's hope for the latter!

Comments

@qgib
Copy link
Contributor

qgib commented Jan 30, 2014

Author Name: Etienne Tourigny (@etiennesky)
Original Redmine Issue: 9460
Affected QGIS version: 2.0.1

Assignee: Etienne Tourigny

GdalTools should rely on commandline tools only for complex operations, not basic dataset info.

The SRS and extent query methods rely on gdalinfo, which is fragile as reported by Even Rouault:

#d63104ae50dcd45ddf50dfe31ea6803530a64b26#commitcomment-5218484

Just an outsider point of view : why using GDAL utilties and risky output parsing to get the SRS and not just plain Python GDAL API...

I have a pull request to fix this, but I am hesitant to apply this to master now. Any input would be appreciated.
@qgib
Copy link
Contributor Author

qgib commented Jan 30, 2014

Author Name: Etienne Tourigny (@etiennesky)

see pull request #1129

@qgib
Copy link
Contributor Author

qgib commented Feb 5, 2014

Author Name: Giuseppe Sucameli (@brushtyler)

Fixed in changeset "58faa5b4ad113860e798f2b5fdeb0741bb864e09".

  • status_id was changed from Open to Closed

@qgib
Copy link
Contributor Author

qgib commented Feb 5, 2014

Author Name: Giuseppe Sucameli (@brushtyler)

Merged, thanks Etienne.

  • resolution was changed from to fixed/implemented

@qgib qgib added the Bug Either a bug report, or a bug fix. Let's hope for the latter! label May 24, 2019
@qgib qgib closed this as completed May 24, 2019
@qgib qgib mentioned this issue May 25, 2019
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Bug Either a bug report, or a bug fix. Let's hope for the latter!
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@qgib