-
Notifications
You must be signed in to change notification settings - Fork 6
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
Fix overwrite logic when downloading granule #33
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
bsatoriu
approved these changes
May 12, 2022
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great, thank you!
chuckwondo
added a commit
that referenced
this pull request
Oct 4, 2022
chuckwondo
added a commit
that referenced
this pull request
Oct 4, 2022
chuckwondo
added a commit
that referenced
this pull request
Oct 4, 2022
bsatoriu
added a commit
that referenced
this pull request
Oct 25, 2023
* Edl federated login (#20) * Edl federated login * AWS class * Add optional expiration param to aws methods * DPS EDL login support * DPS EDL login testing * DPS EDL login testing * DPS EDL login testing * maap.aws.earthdata_s3_credentials() * Update bucket for uploadFiles (#23) * Update bucket for uploadFiles * revert change to path * Make Result.getData raise on failed HTTP download In addition: - Ignore Python-, OS-, and IDE-specific files - Remove accidentally committed checkpoint file - Tidy README.md formatting - Bump version to 0.4.0 * Fix overwrite logic when downloading granule (#33) Fixes #26 * Add the DAAC url to the earthdata_s3_credentials output * Apply hotfixes (#42) Co-authored-by: bsatoriu <[email protected]> Co-authored-by: Aimee Barciauskas <[email protected]> Co-authored-by: Jamison French <[email protected]> * Introduce job model to maap-py from sister/imgspec (#45) * Prepare for v3.0.0 (#46) * Develop (#22) Add optional expiration param to aws methods * Update bucket for uploadFiles (#23) * Update bucket for uploadFiles * revert change to path * maap.aws.earthdata_s3_credentials() * Merge hotfixes into master branch (#43) Co-authored-by: Jamison French <[email protected]> * Create LICENSE * Prepare for v3.0.0 --------- Co-authored-by: bsatoriu <[email protected]> Co-authored-by: Aimee Barciauskas <[email protected]> Co-authored-by: Chuck Daniels <[email protected]> Co-authored-by: Jamison French <[email protected]> Co-authored-by: Marjorie Lucas <[email protected]> * Change log level and add try catch when submitting job (#47) * Clean up logging (#49) MAAP-Project/Community#773 * chore: remove query service * Create maap.downloadGranule() method and unit test (#52) * Dont retrieve attributes after job submission to reduce api queries (#48) * Add functionality to cancel jobs (#53) * Add maap.aws.workspace_bucket_credentials() (#54) --------- Co-authored-by: Aimee Barciauskas <[email protected]> Co-authored-by: Chuck Daniels <[email protected]> Co-authored-by: Jamison French <[email protected]> Co-authored-by: Sujen Shah <[email protected]> Co-authored-by: Marjorie Lucas <[email protected]> Co-authored-by: Jamison French <[email protected]>
bsatoriu
added a commit
that referenced
this pull request
Oct 25, 2023
* Edl federated login (#20) * Edl federated login * AWS class * Add optional expiration param to aws methods * DPS EDL login support * DPS EDL login testing * DPS EDL login testing * DPS EDL login testing * maap.aws.earthdata_s3_credentials() * Update bucket for uploadFiles (#23) * Update bucket for uploadFiles * revert change to path * Make Result.getData raise on failed HTTP download In addition: - Ignore Python-, OS-, and IDE-specific files - Remove accidentally committed checkpoint file - Tidy README.md formatting - Bump version to 0.4.0 * Fix overwrite logic when downloading granule (#33) Fixes #26 * Add the DAAC url to the earthdata_s3_credentials output * Apply hotfixes (#42) * Introduce job model to maap-py from sister/imgspec (#45) * Prepare for v3.0.0 (#46) * Develop (#22) Add optional expiration param to aws methods * Update bucket for uploadFiles (#23) * Update bucket for uploadFiles * revert change to path * maap.aws.earthdata_s3_credentials() * Merge hotfixes into master branch (#43) * Create LICENSE * Prepare for v3.0.0 --------- * Change log level and add try catch when submitting job (#47) * Clean up logging (#49) MAAP-Project/Community#773 * chore: remove query service * Create maap.downloadGranule() method and unit test (#52) * Dont retrieve attributes after job submission to reduce api queries (#48) * Add functionality to cancel jobs (#53) * Add maap.aws.workspace_bucket_credentials() (#54) --------- Co-authored-by: Aimee Barciauskas <[email protected]> Co-authored-by: Chuck Daniels <[email protected]> Co-authored-by: Jamison French <[email protected]> Co-authored-by: Sujen Shah <[email protected]> Co-authored-by: Marjorie Lucas <[email protected]> Co-authored-by: Jamison French <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fixes #26