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 Time Series Workflow #47

Merged
merged 54 commits into from
Sep 23, 2024
Merged

Add Time Series Workflow #47

merged 54 commits into from
Sep 23, 2024

Conversation

AndrewPlayer3
Copy link
Contributor

@AndrewPlayer3 AndrewPlayer3 commented Sep 17, 2024 •
edited
Loading

Confirmed that output still matches after changes.

AndrewPlayer3 and others added 30 commits September 4, 2024 13:11
@AndrewPlayer3
time series entrypoint
6d1fc13
@AndrewPlayer3
add entrypoint to __main__
ec782b5
@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
@jacquelynsmale
add - to timeseries
bbb19da
@AndrewPlayer3
pick only 1 bbox and run with it
1ee8440
@AndrewPlayer3
Merge branch 'timeseries-entrypoint' of https://github.com/asfhyp3/hy…
1624608 
…p3-srg into timeseries-entrypoint
@AndrewPlayer3
create product name for timeseries
86ffc99
@AndrewPlayer3 AndrewPlayer3 marked this pull request as ready for review September 19, 2024 15:25
@AndrewPlayer3 AndrewPlayer3 requested a review from a team as a code owner September 19, 2024 15:25
forrestfwilliams
forrestfwilliams reviewed Sep 19, 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.

Core functionality is great. Just some cleaning up and testing to do.

Dockerfile Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
src/hyp3_srg/back_projection.py Show resolved Hide resolved
src/hyp3_srg/back_projection.py Outdated Show resolved Hide resolved
src/hyp3_srg/time_series.py Show resolved Hide resolved
src/hyp3_srg/time_series.py Outdated Show resolved Hide resolved
src/hyp3_srg/time_series.py Outdated Show resolved Hide resolved
src/hyp3_srg/time_series.py Outdated Show resolved Hide resolved
src/hyp3_srg/time_series.py Show resolved Hide resolved
forrestfwilliams
forrestfwilliams reviewed Sep 23, 2024
View reviewed changes
README.md Outdated Show resolved Hide resolved
@AndrewPlayer3 @forrestfwilliams
Update README.md
f512313 
Co-authored-by: Forrest Williams <[email protected]>
@AndrewPlayer3 AndrewPlayer3 merged commit 32caf3f into develop Sep 23, 2024
8 checks passed
@AndrewPlayer3 AndrewPlayer3 deleted the timeseries-entrypoint branch September 23, 2024 17:46
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
None yet
Projects
None yet
Milestone
No milestone
Development

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