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

Add Landsat 1-5 (MSS and TM) and 7 (ETM) Item creation #13

Merged
merged 47 commits into from
Mar 22, 2022
Merged

Add Landsat 1-5 (MSS and TM) and 7 (ETM) Item creation #13

merged 47 commits into from
Mar 22, 2022

Conversation

pjhartzell
Copy link
Contributor

Adds functions (all in usgs_stac.py) to create new STAC Items from one or more USGS stac_version: 1.0.0 Items.

  • Uses same input as current landsat-8 method (a single *_MTL.xml file href)
  • Tested on an MSS (Landsat 1-5, Collection 2, Level-1) xml
  • Tested on a TM (Landsat 4-5, Collection 2, Level-2) xml
  • Note the tests currently create the example MSS and TM STAC items

@pjhartzell pjhartzell requested a review from gadomski February 18, 2022 22:11
gadomski
gadomski requested changes Feb 22, 2022
View reviewed changes
examples/usgs-stac-convert/mss/LM01_L1GS_001010_19720908_02_T2.json Outdated Show resolved Hide resolved
examples/usgs-stac-convert/mss/LM01_L1GS_001010_19720908_02_T2.json Outdated Show resolved Hide resolved
tests/test_usgs_stac.py Outdated Show resolved Hide resolved
tests/test_usgs_stac.py Outdated Show resolved Hide resolved
@pjhartzell pjhartzell changed the title Feature/landsat4 5 Add Landsat 1-5 (MSS and TM) and 7 (ETM) Item creation Feb 27, 2022
@pjhartzell pjhartzell requested a review from gadomski February 28, 2022 14:51
gadomski
gadomski requested changes Feb 28, 2022
View reviewed changes
src/stactools/landsat/stac.py Outdated Show resolved Hide resolved
src/stactools/landsat/stac.py Outdated Show resolved Hide resolved
src/stactools/landsat/stac.py Outdated Show resolved Hide resolved
src/stactools/landsat/stac.py Outdated Show resolved Hide resolved
src/stactools/landsat/stac.py Outdated Show resolved Hide resolved
@pjhartzell pjhartzell force-pushed the feature/landsat4-5 branch from 515315f to 3509796 Compare March 3, 2022 20:43
@pjhartzell pjhartzell requested a review from gadomski March 7, 2022 14:19
@pjhartzell
sundry review edits
a95e9be 
- switch legacy_l8 command option to a flag, which changes the command default for using the legacy l8 method to to False
- remove "deprecated" terminology
- grammar edit
@pjhartzell
make asset hrefs absolute when creating assets
99a3e47 
- allows the hrefs to then be made relative
@pjhartzell
remove relative asset paths from item command, import formatting
50e126d
@pjhartzell pjhartzell requested a review from gadomski March 8, 2022 13:33
@gadomski gadomski mentioned this pull request Mar 18, 2022
Merged
5 tasks
@pjhartzell
chore: clean up .gitignore
cc52805
@pjhartzell
chore: change stactools dependency to point at commit hash rather tha…
8e1b93b 
…n branch
@pjhartzell
chore: update docs
856f972 
- update ipynb
- remove the legacy l8 item directory from examples directory
- update command function documentation
@pjhartzell
chore: update changelog
6de9fe1
@pjhartzell pjhartzell requested a review from gadomski March 21, 2022 20:18
@pjhartzell pjhartzell marked this pull request as ready for review March 21, 2022 20:34
gadomski
gadomski reviewed Mar 21, 2022
View reviewed changes
.gitignore Outdated Show resolved Hide resolved
@pjhartzell pjhartzell merged commit 5cfff89 into stactools-packages:main Mar 22, 2022
@pjhartzell pjhartzell deleted the feature/landsat4-5 branch March 22, 2022 18:03
This was referenced Mar 22, 2022
Closed
Closed
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
@pjhartzell @gadomski