-
Notifications
You must be signed in to change notification settings - Fork 40
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
Create a Zenodo download manager #697
Conversation
We might want to update |
Would you recommend using |
import requests | ||
import wget | ||
|
||
url = "https://zenodo.org/api/records/" |
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.
Same here. Not sure whether we should put it in a const module.
I'm about to say the same thing since it's in STL. |
We can do this with requests too:
|
Done |
10345b6
to
763b1ac
Compare
The update of the |
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.
Nice
Pull Request doc
Purpose
Build the
Zenodo
class around the Zenodo Rest API to efficiently handle data coming from different record. Partially addresses #687.What the code is doing
Testing
Manual testing.
Where to look
Zenodo
class inpowersimdata.network.zenodo
moduleTUB
class located in thepowersimdata.network.europe_tub.model
modulezenodo_get
package from Pipfile and requirements.txtUsage Example/Visuals
We can now do that:
Time estimate
30min