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

feat: TDE-452 create imagery stac item #100

Closed
wants to merge 24 commits into from
Closed

Conversation

MDavidson17
Copy link
Contributor

@MDavidson17 MDavidson17 commented Aug 22, 2022

  • stac item is created based on a similar implementation in basemaps
  • pystac was only used to get the stac version, this can be hardcoded and pystac removed entirely, or the code can be changed to return a pystac item object. -pystac removed

nb: The code has been implemented with thought of how the create_stac_items.py script might run for different datasets.
How it is currently written it could be possible to add a datatype dropdown input in the argo workflow and use this datatype to determine what stac to create.
In topo-processor this was premtively scripted, however on reflection the developers mentioned they wouldn't have implemented it until there were additional datasets, therefore it has not been implemented yet in topo-imagery.

@MDavidson17 MDavidson17 marked this pull request as ready for review August 22, 2022 02:30
@MDavidson17 MDavidson17 requested a review from a team as a code owner August 22, 2022 02:30
@MDavidson17 MDavidson17 marked this pull request as draft August 22, 2022 20:14
@MDavidson17 MDavidson17 marked this pull request as ready for review August 28, 2022 22:49
scripts/stac/imagery_stac.py Outdated Show resolved Hide resolved
@MDavidson17 MDavidson17 marked this pull request as draft September 4, 2022 22:21
@MDavidson17
Copy link
Contributor Author

this work has been superceeded by:
#133
#132
#134
#135

@MDavidson17 MDavidson17 closed this Sep 9, 2022
@MDavidson17 MDavidson17 deleted the feat/create_stac branch September 9, 2022 04:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants