search

OpenDataAnalytics / gaia

Gaia is a geospatial analysis library jointly developed by Kitware and Epidemico.
31 stars 15 forks source link

API Rewrite: Create a cleaner, more flexible API for IO and processes #117

Closed scottwittenburg closed 6 years ago

scottwittenburg commented 6 years ago

So far, only crop is supported for raster and pandas datasets. Other features are missing at the moment as well, including filtering, geo transforms, and dataset writers.

coveralls commented 6 years ago

Coverage Status

Coverage decreased (-24.8%) to 48.302% when pulling d7c0a05b005b3933e19afc1377fabf5f1ac73a10 on api-rewrite into 4737c040baa8b0efd34a6b15eee62e71d4530743 on master.

scottwittenburg commented 6 years ago

Sounds fine to me, do you want me to merge it?

johnkit commented 6 years ago

Yep, go ahead and merge, Scott. (Or if there's a permission issue, let me and Aashish know.)

scottwittenburg commented 6 years ago

I'm just remembering that this would be the first merge to master changing API (and removing much of the functionality), which I think is why I labeled this PR with a WIP. Are we sure we're ready to merge @johnkit and @aashish24?

aashish24 commented 6 years ago

@scottwittenburg I believe if you have tagged the current master as old-master, then yes, please go ahead remove the WIP and merge it into master :+1:

scottwittenburg commented 6 years ago

Yes, it's tagged master-pre-api-rewrite for future reference. I'll merge this.