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
Added RasterSource/GDALBand GDAL objects #3694
Conversation
6200276
to
e2c5feb
Compare
|
||
class GDALBand(GDALBase): | ||
""" | ||
Wraps a GDAL raster band, needs to be instantiated from a RasterSource object. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
from a -> with a?
bc40ace
to
dfffae2
Compare
I've rebased/merged all the latest reviews. |
@yellowcap Are you planning a second review after the changes I introduced? Do you find them satisfying? |
PARAMETER["false_easting",400000], | ||
PARAMETER["false_northing",0], | ||
UNIT["metre",1, | ||
AUTHORITY["EPSG","9001"]]] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should probably be updated with the new gdalinfo, which basically includes one more line in the coordinate system:
AUTHORITY["EPSG","3086"]]
Just made a few minor comments. Other than those small things I think its good to go. @claudep - the latest updates you made are statisfying, thanks for those! |
Based on Daniel Wiesmann's raster branch. Thanks Daniel Wiesmann and Tim Graham for the reviews. Refs #23804.
Unfortunately Jenkins keeps giving unexplained errors, let's not consider its results for this commit. |
Merged in 6e08bde |
Based on Daniel Wiesmann's raster branch. Refs #23804.