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.

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

Add DelayedMultibandTile #223

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

echeipesh
Copy link
Collaborator

Closes: #199

Trying to integrate this into RasterRegion raises more questions:

Currently signature of RasterRegion.raster: Option[Raster[MultibandTile]] seems very clunky considering that current implementation of RasterRegion does not allow creating region that does not intersect source and the that expected behavior is for raster region to buffer out missing pixels. The implication seems to be that RasterRegion will do its best to produce an expected tile. So the Option part doesn't make so much sense, only exceptional failures might prevent production of the result.

Something that delegates reading to a function and caches the result
@echeipesh echeipesh self-assigned this Aug 5, 2019
@CloudNiner CloudNiner changed the base branch from master to develop September 25, 2019 01:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

RasterRegion should produce LazyMultibandTile
1 participant