Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # 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
# for free to join this conversation on GitHub. Already have an account? # 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