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

loose the limit of only processing the NADIR scenes #42

Merged
merged 33 commits into from
Apr 11, 2024
Merged

loose the limit of only processing the NADIR scenes #42

merged 33 commits into from
Apr 11, 2024

Conversation

cirrusasf
Copy link
Contributor

No description provided.

@cirrusasf cirrusasf added the bumpless Changes to documentation, CI/CD pipelines, etc that don't affect the project's version label Apr 4, 2024
@cirrusasf cirrusasf marked this pull request as ready for review April 4, 2024 22:13
@cirrusasf cirrusasf requested a review from a team as a code owner April 4, 2024 22:13
@cirrusasf cirrusasf added the patch Bump the patch version number of this project label Apr 4, 2024
@cirrusasf cirrusasf removed the bumpless Changes to documentation, CI/CD pipelines, etc that don't affect the project's version label Apr 4, 2024
jhkennedy
jhkennedy reviewed Apr 5, 2024
View reviewed changes
tests/landsat/test_main.py Show resolved Hide resolved
tests/landsat/test_main.py Outdated Show resolved Hide resolved
tests/landsat/test_main.py Outdated Show resolved Hide resolved
CHANGELOG.md Outdated Show resolved Hide resolved
landsat/src/__init__.py Outdated Show resolved Hide resolved
tests/landsat/test_main.py Outdated Show resolved Hide resolved
landsat/src/main.py Outdated Show resolved Hide resolved
landsat/src/main.py Outdated Show resolved Hide resolved
cirrusasf and others added 15 commits April 5, 2024 16:16
@cirrusasf @jhkennedy
Update CHANGELOG.md
cd637bd 
Co-authored-by: Joseph H Kennedy <[email protected]>
@cirrusasf @jhkennedy
Update tests/landsat/test_main.py
5cfe39d 
Co-authored-by: Joseph H Kennedy <[email protected]>
@cirrusasf
modify test function code to adopt unittest mock class
1c9b874
@cirrusasf
remove unused data files
3882759
@AndrewPlayer3
modified test to use hyp3-sdk rather than json
9d4831c
@AndrewPlayer3
remove unnecessary json files
15ffb4f
@cirrusasf
add test_submit_pairs_for_processing function to test_main.py
173a2a4
@cirrusasf
remove code for debug
db9a3dd
@cirrusasf
modify main.py code
697b880
@cirrusasf
ruff format test_main.py
2fc4010
@cirrusasf
make variable name more sense
423743a
@AndrewPlayer3
Merge branch 'handle_off_nadir_scenes' of https://github.com/ASFHyP3/…
c10628d 
…its-live-monitoring into off_nadir_andrew
@AndrewPlayer3
formatting
673457c
@AndrewPlayer3
formatting
c49a67f
@AndrewPlayer3
Merge pull request #54 from ASFHyP3/off_nadir_andrew
55497ef 
Modified Test to Use hyp3_sdk Instead of a JSON File
AndrewPlayer3
AndrewPlayer3 reviewed Apr 10, 2024
View reviewed changes
tests/landsat/test_main.py Outdated Show resolved Hide resolved
tests/landsat/test_main.py Outdated Show resolved Hide resolved
tests/landsat/test_main.py Outdated Show resolved Hide resolved
@cirrusasf cirrusasf merged commit a8afa89 into develop Apr 11, 2024
4 checks passed
@cirrusasf cirrusasf deleted the handle_off_nadir_scenes branch April 11, 2024 19:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jhkennedy jhkennedy jhkennedy left review comments

@AndrewPlayer3 AndrewPlayer3 AndrewPlayer3 approved these changes

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

Successfully merging this pull request may close these issues.
3 participants
@cirrusasf @jhkennedy @AndrewPlayer3