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

WIP: Feature Request: add Multiband support to geotrellis.spark.resample ZoomResample #2511

Closed

Conversation

aklink
Copy link
Contributor

@aklink aklink commented Dec 12, 2017

Feature Request: add MultibandTile support to geotrellis.spark.resample ZoomResample

geotrellis.spark.resample ZoomResample uses TileLayerRDD[K], only supporting RDD[(K,Tile)] with Metadata[TileLayerMetadata[K]]

This Pull Request / Feature Request has the purpose to add MultibandTile support to it by using MultibandTileLayerRDD[K] to RDD[(K,MultibandTile)] with Metadata[TileLayerMetadata[K]]

Current approach is adding ZoomResampleMultiband and ZoomResampleMultibandMethods to geotrellis.spark.resample and extend Implicits by implicit class withZoomResampleMultibandMethods[K: SpatialComponent](self: MultibandTileLayerRDD[K]) extends ZoomResampleMultibandMethods[K](self)

ToDo's:

  • Test: add ZoomResampleMultibandSpec.scala in src/test/scala/geotrellis/spark/resample
  • Improve: combine ZoomResample and ZoomResampleMultiband into one file by using RDD[(K,V)] with Metadata[TileLayerMetadata[K]] with V supporting both Tile and MultibandTile

…LayerRDD

Signed-off-by: Adrian Klink <Adrian.Klink@eftas.com>
… MultibandTileLayerRDD

Signed-off-by: Adrian Klink <Adrian.Klink@eftas.com>
@aklink
Copy link
Contributor Author

aklink commented Feb 26, 2018

solved by commit 3dbcc4f

@aklink aklink closed this Feb 26, 2018
@aklink
Copy link
Contributor Author

aklink commented Feb 26, 2018

see #2569

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant