-
Notifications
You must be signed in to change notification settings - Fork 5
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
EarthDataDownloader object to retain user login credentials #200
Comments
If you're not familiar with it, I strongly recommend earthaccess. It's a great way to manage Earthdata logins. The library I maintain, icepyx has it set up as an authentication MixIn class so the credentials are accessible from any module that needs them but only asks users for them once. |
Thanks for the suggestions! I had a look, but I'm not sure if this will work here. For now, I've kept it as is, but added better documentation for how to set temporarily or permanent environment variables so users don't need to continuously supply the login credentials. |
I think as long as it is well documented then this is enough to resolve the issue |
Ok that sounds good to me, I'll close this as I've added better documentation in #212 in this how-to guide. |
It's not part of the user api (and probably not documented), but with an earthaccess Auth object (created with As @PennyHow noted - not required for JOSS, but wanted to follow up and answer your question! |
Ahh great! Thanks for finding that, this is working to access the credentials: I'll add this into PolarToolkit to replace the custom class I had. |
Description of the desired feature: Functionality added so that login credentials are retained by the
EarthDataDownloader
object after first inputted, and therefore the user does not have to keep re-defining them for each download.In the tutorials, there is repeated EarthData login requests. For example, in the 3D_stack tutorial:
Also, there is nothing in your documentation on how to define credentials as environment variables (the icepack link in the API reference for EarthDataDownloader is broken).
The text was updated successfully, but these errors were encountered: