search

GeoscienceAustralia / agdc

Repository for Australian Geoscience Data Cube (AGDC) code
BSD 3-Clause "New" or "Revised" License
29 stars 24 forks source link

AGDC API #54

Closed simonoldfield closed 9 years ago

simonoldfield commented 9 years ago

Merge the AGDC API as it stands back into the develop branch

wenjunwu commented 9 years ago

There is a question regarding this API. Can we comment on that if the API can be used for MODIS data, and what additional work might be required in future if not.

simonoldfield commented 9 years ago

The reality is that the current API has only been tested in terms of it's support of Landsat, Landsat derived (e.g. Water Observations From Space), and DEM/DSM datasets.

In theory it should just be a matter of defining the correct reference data (enums and constants and so on) for MODIS to work. BUT, due to resource constraints and somewhat limited access to MODIS test data, this has not been done/tested as yet.

The other factor is that there is work currently in progress to move the database and logical model to being more generalised and less earth observation specific. This will have flow on impacts on the API so a deliberate decision was made not to pre-empt these changes and potentially do work that will need to be re-done, and similarly to try to limit the impact to downstream consumers of the API to one breaking change.

wenjunwu commented 9 years ago

Thanks for the clarification. I’m ok with the merge so that it would be easier to continue the development as described.