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.

Sign up for GitHub

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

Jump to bottom

PC Backports Sen3: asset and item properties #15

Merged
merged 4 commits into from
Jun 13, 2023
Merged

PC Backports Sen3: asset and item properties #15

merged 4 commits into from
Jun 13, 2023

Conversation

ircwaves
Copy link
Contributor

Description:

  • convert item properties to snake-case
  • use correct STAC EO Extension attributes
  • set datetime entries in ISO 8601 format

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.
  • Changes are added to the CHANGELOG.

ircwaves added 4 commits June 13, 2023 09:44
@ircwaves
update asset extra fields to s3-pseudo-extension style
cc90061 
- drop 'license' from items (it should be at the collection level)
- drop 'local_path'
- translate sral:bands to s3:altimetry_bands
- transform spectral characteristics from nm to um
@ircwaves
item properties refactored to lowercase w/ underscores
9aabe24
@ircwaves
use xml timestamp instead of filename timestamp
7e34bf6
@ircwaves
update CHANGELOG.md
02b9314
@ircwaves ircwaves requested a review from gadomski June 13, 2023 14:14
@ircwaves ircwaves merged commit 255e810 into stactools-packages:main Jun 13, 2023
@ircwaves ircwaves deleted the pc-backports-sentinel3 branch June 13, 2023 22:07
@ircwaves ircwaves mentioned this pull request Jun 15, 2023
Merged
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@gadomski gadomski gadomski approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@ircwaves @gadomski