search

locationtech / geotrellis

GeoTrellis is a geographic data processing engine for high performance applications.
http://geotrellis.io
Other
1.33k stars 360 forks source link

API renames #1267

Closed lossyrob closed 8 years ago

lossyrob commented 8 years ago

Prior to the 0.10 release, we want to make some renames to core types so that they are properly named before the release. This will break the API and any any client code pretty heavily, so this should be a one-shot thing.

MultiBand -> Multiband
MetaData -> Metadata
SpatialKey -> GridKey
SpatialComponent -> GridComponent
SpaceTimeKey -> TimeGridKey
RasterRDD[K] -> TileRDD[K]
MultiBandRasterRDD[K] -> MultiBandTileRDD[K]

Moving op to mapalgebra, and moving any operations that are not map algebra to a different subpackage in the root package (e.g. geotrellis.raster.op.local.LocalMethods.mask(ext: Extent, geoms: Traversable[Geometry], includeExterior: Boolean) should be functionality in some geotrellis.raster.mask package)

lossyrob commented 8 years ago

@echeipesh @moradology @pomadchin @jamesmcclain @dwins can you think of others? Any feedback on these?

pomadchin commented 8 years ago

Nice catch with key types naming (though may be because of legacy GridTimeKey also looks nice)

echeipesh commented 8 years ago

+1 on class renames.

Less certain on the op -> mapalgebra move. What is the motivation, flatten the namespace a little? Are things in raster package going to mirror this move?

lossyrob commented 8 years ago

Fixed by https://github.com/lossyrob/geotrellis/pull/12