Ryan Grout
Ryan Grout
I can make the check use transforms. Should it live in the transform.py module or stay in coords.py? Also, I'm trying to dig into why rowcol transforms, especially for scalar...
In looking at this some more, I realized that `TransformMethodsMixin` has an `.index()` method which will return the the row, col coordinates of a point. Why this isn't called rowcol...
@sgillies Are you thinking something more like this? ```python def contains_coord(transform, height, width, coords): """Test if coord is contained in bounds. Coordinate and bounds are assumed to be relative to...
@snowman2 After looking over this PR some more, I realized there is some other subtle issues with `get_minimum_dtype`. 1. Bool should be return if range [0, 1] (was returning uint8)...
> @groutr, I think you have raised some valid points. I think the inf/nan changes are good to go. However, the empty array logic appears to need more thought/discussion. Mind...
@sgillies There are several questions in this PR that need guidance/clarification. 1. What is the proper return for empty values? I feel None is the appropriate return value. Bool is...
An elaboration of what making rasterio dtypes explicit might look like. Welcome to any feedback. This might not be enough of an issue to justify the effort, however, some good...
> @snowman2 @groutr (adding you since you're a key merge contributor) if we have a virtual merging method like this one, what would you think about it returning a "Virtual...
> A virtual dataset could be a thin wrapper around an XML bytestring. We might consider adding internal validation against the VRT schema (https://raw.githubusercontent.com/OSGeo/gdal/master/data/gdalvrt.xsd). xmlschema seems like a nice package...
Is the goal of the VirtualDataset to replicate the functionality of a rasterio Dataset, permit construction and then be able to render a vrt xml or is it simply serving...