Minor earthaccess.download
updates
#317
Merged
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.
This PR contains a few ergonomic updates to
earthaccess.download
. Specifically:earthaccess.download
now accepts a single granule as input in addition to a list of granulesearthaccess.download
are now the fully qualified local paths. This makes it easier to do things with the paths that are returned (currently there is some extraos.path.join
that needs to happen)local_path
can beNone
when usingearthaccess.download
. We weren't accounting for that possibility in all the possibledownload
cases (e.g. downloading from S3). This PR moves thelocal_path=None
logic higher up in the call stack to account for these cases.