Repository navigation
Add interpretAs and withNoData methods to Tile - #1702
Conversation
Allows for more flexibility when working around NoData values
It was dropping NoDataHandling portions of CellType
|
Is this WIP until the RFC gets worked out? |
|
I'd be happy to bundle them together, but they don't have to be. This PR can be merged as is and addresses the issue that spawned it. |
b9c3a05 to
6f3a2ab
Compare
|
@lossyrob @moradology Ready for review again after One final through is that what is currently
|
|
@echeipesh I think |
|
I also think |
|
I prefer
|
|
Semantic ambiguity could be addressed in docstrings. |
|
+1 |
|
I like the word
|
|
Thank you everybody for chiming in on the naming! |
Fixes: #1674
Allows for more flexibility when working around NoData values.
Additionally this expands the capacity of the
ConstantTileto represent constant tiles other than tiles withConstantNoDataHandlingto satisfy the conversions implied by the new API.TODO:
Tile.withNoDatanoDataValueconstructor overload