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.

Sign up for GitHub

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

Jump to bottom

RasterSourceRDD resample + overview params missing #3216

Open
moradology opened this issue Mar 31, 2020 · 1 comment
Open

RasterSourceRDD resample + overview params missing #3216

moradology opened this issue Mar 31, 2020 · 1 comment
Assignees
@echeipesh
Labels
bug

Comments

@moradology
Copy link
Contributor

In the course of resolving pr #3200, a few small bugs were detected that could be traced back to the RasterSourceRDD. Various RasterSourceRDD creation methods exist (e.g. spatial and temporal), but there's no consistency in the usage of OverviewStrategy and ResampleMethod. The upshot of this is that OverviewStrategy and ResampleMethod instances that the user would like to use for selecting which overview to source and determining how to sample said source imagery are ignored
@moradology moradology added the bug label Mar 31, 2020
CloudNiner added a commit to moradology/geotrellis that referenced this issue Mar 31, 2020
@CloudNiner
Fix broken GDALRasterSourceRDDSpec
fdfc6cb 
Same issue as in previous commit for
RasterSourceRDDSpec, need to provide strategy.

See: locationtech#3216
@CloudNiner
Copy link
Contributor

#3200 adds support for OverviewStrategy and ResampleMethod parameters to the RasterSourceRDD.spatial, RasterSourceRDD.temporal and RasterSourceRDD.apply methods. The other RasterSourceRDD methods still need to be addressed.

CloudNiner added a commit to moradology/geotrellis that referenced this issue Apr 1, 2020
@CloudNiner
Fix broken GDALRasterSourceRDDSpec
d88c12f 
Same issue as in previous commit for
RasterSourceRDDSpec, need to provide strategy.

See: locationtech#3216
@echeipesh echeipesh self-assigned this Apr 2, 2020
echeipesh pushed a commit that referenced this issue Apr 6, 2020
@CloudNiner @echeipesh
Fix broken GDALRasterSourceRDDSpec
ed5d2e3 
Same issue as in previous commit for
RasterSourceRDDSpec, need to provide strategy.

See: #3216
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@echeipesh echeipesh

Labels
bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@echeipesh @CloudNiner @moradology