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

Release v0.8.0 #48

Merged
merged 65 commits into from
Sep 23, 2024
Merged

Release v0.8.0 #48

merged 65 commits into from
Sep 23, 2024

Conversation

AndrewPlayer3
Copy link
Contributor

@AndrewPlayer3 AndrewPlayer3 commented Sep 23, 2024 •
edited
Loading

image

forrestfwilliams and others added 30 commits August 29, 2024 16:00
@forrestfwilliams
update readme
e49e11c
@AndrewPlayer3
time series entrypoint
6d1fc13
@AndrewPlayer3
add entrypoint to __main__
ec782b5
@forrestfwilliams
Merge pull request #44 from ASFHyP3/readme
fa0f180 
Update readme
@AndrewPlayer3
Merge branch 'develop' of https://github.com/asfhyp3/hyp3-srg into ti…
93f044c 
…meseries-entrypoint
@jacquelynsmale
add in basics for running timeseries
67e75ef
@AndrewPlayer3
move create_param_file to utils
3c00346
@AndrewPlayer3
get bboxs and dem
0d61ab1
@AndrewPlayer3
get_bbox and create_param_file
7b9dc8e
@AndrewPlayer3
Merge branch 'timeseries-entrypoint' of https://github.com/asfhyp3/hy…
21c1e1d 
…p3-srg into timeseries-entrypoint
@AndrewPlayer3
next todos
0e2e779
@jacquelynsmale
add in more info for each job type
b160d70
@AndrewPlayer3
added all steps from sentinel_timeseries
b352ed4
@AndrewPlayer3
Merge branch 'timeseries-entrypoint' of https://github.com/asfhyp3/hy…
56e70cd 
…p3-srg into timeseries-entrypoint
@AndrewPlayer3
timeseries to time_series
1381c9e
@AndrewPlayer3
os to shutil
f3da67e
@AndrewPlayer3
comments
e731dd4
@AndrewPlayer3
.open to width
4287d21
@AndrewPlayer3
formatting
f23899c
@AndrewPlayer3
various fixes and cleanup
d0466ed
@AndrewPlayer3
copy snaphu to the correct dir
e45865e
@forrestfwilliams
fix formatting
2e627a1
@forrestfwilliams
update product loading
db309b5
@forrestfwilliams
further product loading updates
4ee738c
@forrestfwilliams
update packaging
a298753
@AndrewPlayer3
Merge pull request #45 from ASFHyP3/forrest_edits
98ab3f8 
Add product loading and packaging to time_series tool
@AndrewPlayer3
fixed name
2ff749f
@AndrewPlayer3
unused import
8f07db1
@AndrewPlayer3
add elevation.dem.rsc to product
e99dca1
@AndrewPlayer3
flake8
e997bd6
AndrewPlayer3 and others added 16 commits September 19, 2024 09:31
@AndrewPlayer3
updaged changelog
675a8e5
@AndrewPlayer3
refactoring for using bounds
61ef231
@AndrewPlayer3
negative cases for dem download
7358e60
@AndrewPlayer3
refactoring
2de20d5
@AndrewPlayer3 @forrestfwilliams
Update README.md
af689a1 
Co-authored-by: Forrest Williams <[email protected]>
@AndrewPlayer3
add snaphu
172aa59
@AndrewPlayer3
doc strings
78c9ecd
@AndrewPlayer3
updated readme
a9fa06b
@AndrewPlayer3 @forrestfwilliams
Update src/hyp3_srg/time_series.py
3096321 
Co-authored-by: Forrest Williams <[email protected]>
@AndrewPlayer3
shapely bounds instead of stanford bounds
aa9ddeb
@AndrewPlayer3
updated tests
c61e6f2
@AndrewPlayer3
Merge branch 'timeseries-entrypoint' of https://github.com/asfhyp3/hy…
f9668ff 
…p3-srg into timeseries-entrypoint
@AndrewPlayer3
flake8
aa73b40
@AndrewPlayer3
unit test for getting s3 download arguments
454c3f7
@AndrewPlayer3 @forrestfwilliams
Update README.md
f512313 
Co-authored-by: Forrest Williams <[email protected]>
@AndrewPlayer3
Merge pull request #47 from ASFHyP3/timeseries-entrypoint
32caf3f 
Add Time Series Workflow
@AndrewPlayer3 AndrewPlayer3 added the minor Bump the minor version number of this project label Sep 23, 2024
@AndrewPlayer3 AndrewPlayer3 requested a review from a team as a code owner September 23, 2024 18:45
@github-actions GitHub Actions
Copy link

github-actions bot commented Sep 23, 2024 •
edited by forrestfwilliams
Loading

Developer checklist

  • Indicated the level of changes to this package by affixing one of these labels:
    • major -- Major changes to the API that may break current workflows
    • minor -- Minor changes to the API that do not break current workflows
    • patch -- Patches and bugfixes for the current version that do not break current workflows
    • bumpless -- Changes to documentation, CI/CD pipelines, etc. that don't affect the software's version
  • (If applicable) Updated the dependencies and indicated any downstream changes that are required
  • Added/updated documentation for these changes
  • Added/updated tests for these changes
  • Verified changes in test deployment and summarized results, e.g. in PR description or comments on the related issue(s)

Reviewer checklist

  • Have all dependencies been updated?
  • Is the level of changes labeled appropriately?
  • Are all the changes described appropriately in CHANGELOG.md?
  • Has the documentation been adequately updated?
  • Are the tests adequate?
  • Have the changes been verified in the test deployment?

@forrestfwilliams
Copy link
Contributor

Just need to add the [0,0,0,0] as a sentinel value for "no bounding box".

@forrestfwilliams forrestfwilliams self-requested a review September 23, 2024 19:02
forrestfwilliams
forrestfwilliams requested changes Sep 23, 2024
View reviewed changes
Copy link
Contributor

@forrestfwilliams forrestfwilliams left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

See above

@forrestfwilliams forrestfwilliams self-requested a review September 23, 2024 19:40
@AndrewPlayer3 AndrewPlayer3 merged commit 7a390fc into main Sep 23, 2024
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@forrestfwilliams forrestfwilliams forrestfwilliams approved these changes

Assignees
No one assigned
Labels
minor Bump the minor version number of this project
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@AndrewPlayer3 @forrestfwilliams @jacquelynsmale