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

Split into COG and NetCDF sub-packages #11

Merged
merged 25 commits into from
Dec 20, 2022
Merged

Split into COG and NetCDF sub-packages #11

merged 25 commits into from
Dec 20, 2022

Conversation

pjhartzell
Copy link
Contributor

@pjhartzell pjhartzell commented Dec 19, 2022

Related Issue(s):
#2

Description:
This is a major reorganization that splits the package into two sub-packages:

  1. netcdf: Collection and Items describing the source NetCDF data files.
  2. cog: Creates tiled COGs from the source NetCDF data files and Items describing each tile of COG data. The Items can optionally contain a derived_from link that points to a STAC Item describing the source NetCDF (e.g., an Item created by the netcdf sub-package).

PR checklist:

  • Code is formatted (run scripts/format).
  • Code lints properly (run scripts/lint).
  • Tests pass (run scripts/test).
  • Documentation has been updated to reflect changes, if applicable.
  • Examples have been updated to reflect changes, if applicable
  • Changes are added to the CHANGELOG.

Add new tests, fill out function docstrings
The version extension is primarily for STAC Item or Collection versions,
not for asset generation methodology versions. The version is now
contained in an extra field on the netcdf Items, which is consistent
with the COG Items.
All class lists are included in the collection item_assets
The class list in the Collection summaries and in Item properties was
ambiguous as to the applicable NetCDF variable. Additional, unlisted
classes also exist in the data that are not part of the class list.
@pjhartzell pjhartzell requested a review from gadomski December 20, 2022 01:18
README.md Outdated Show resolved Hide resolved
environment.yml Outdated Show resolved Hide resolved
setup.cfg Outdated Show resolved Hide resolved
@pjhartzell pjhartzell requested a review from gadomski December 20, 2022 16:50
@pjhartzell pjhartzell merged commit 174dd83 into main Dec 20, 2022
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.

2 participants