search

crim-ca / stac-populator

Workflow logic to populate STAC catalog with demo datasets.
MIT License
2 stars 2 forks source link

Fixes to datacube extension helper #51

Closed dchandan closed 8 months ago

dchandan commented 8 months ago
  1. Some variables (e.g. time_bnds) do not have variable "attributes" in certain model files. As a result the extension code crashes. 8621b6d fixes that.
  2. As per CF convention bounds variables are not required to have attributes and that they should instead be inferred from the corresponding coordinate variables. 6326908 addresses that.

An example of how the data cube extension looks like with these changes is here.

An outstanding issue is that time_bnds variable is still not being inferred as an auxiliary variable. This will require a hack since this variable cannot be determined to be auxiliary using the framework currently in place (because the variable in the netCDF file will not contain any of the "criteria" that could be put into place for the benefit of the is_coordinate).

dchandan commented 8 months ago

@huard your review would be helpful here