Sean Gillies

@brendan-ward tackling just the features module to start is a good idea.

@snowman2 that's a very good point. I think I see an example of that happening here: https://github.com/OSGeo/gdal/blob/35c07b18316b4b6d238f6d60b82c31e25662ad27/ogr/ogrsf_frmts/xls/ogrxlsdatasource.cpp#L116-L118. The code resets the error context, pushes the silencing handler so that no...

I've got another little demo. After changing DatasetReaderBase._read to do this (adding `from cplerr`) ```python except CPLE_BaseError as cplerr: raise RasterioIOError("Read or write failed. {}".format(cplerr)) from cplerr ``` We can...

@snowman2 from your perspective, is there anything unresolved here?

Yeah, we really need to prioritize #1990. It's on my mind for 1.3 and I may even get to do something about it next week.

I think this is a duplicate of #2354 after all. Or not :smile:

@scottyhq if I read https://github.com/OSGeo/gdal/blob/master/apps/gdalwarp_lib.cpp#L613-L614 correctly, vertical grid shift with GDAL 3.4 might be "just" this: `reproject(..., apply_vertical_shift=True)`. The unnamed keyword arguments of reproject get turned into GDAL transformer and...

@scottyhq are you sure you're using the same versions of GDAL, PROJ, its grids, on each side of the comparison? It's easy to get into a place where we have...

``` # This could be a mistake: ipdb> src.transform * src.shape ``` That's invalid use of the affine matrix object on the left. The dataset's shape is `(num_rows, num_cols)`. But...

@daniel-j-h you're right, there's no API module about stacking. But there is a place reserved for it: https://github.com/mapbox/rasterio/pull/1322. And a conceptual framework: https://github.com/mapbox/rasterio/issues/1300.