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.

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

integrate powerplants.csv update into CI #128

Merged
merged 4 commits into from
Jun 11, 2024
Merged

integrate powerplants.csv update into CI #128

merged 4 commits into from
Jun 11, 2024

Conversation

fneum
Copy link
Member

@fneum fneum commented Jul 3, 2023

closes #125

  • restrict that it only runs with new release tags (since it takes a while).
  • check if artifacts get persistent links or if they are deleted after some time
  • if they get deleted, need github action that uploads to zenodo, use REST API, https://developers.zenodo.org/#rest-api

@FabianHofmann
Copy link
Contributor

🚀

@fneum fneum marked this pull request as draft July 25, 2023 12:01
@lkstrp
Copy link
Member

lkstrp commented Jun 11, 2024

Just finishing this:

* [ ]  restrict that it only runs with new release tags (since it takes a while).
* [ ]  check if artifacts get persistent links or if they are deleted after some time
* [ ]  if they get deleted, need github action that uploads to zenodo, use REST API, https://developers.zenodo.org/#rest-
  1. Done
  2. Artifacts do not get a persistent link and are stored for 90 days by default. They are also only available as zip downloads and cannot be loaded directly into img or read via url. That is why I would just use the CI to update the powerplants.csv in the repo. In a potential stats overview (add streamlit application for capacity per country #113) we could just load the file per version from the repo.
  3. If Zenodo is still needed, we can add that. But I don't see why

@lkstrp lkstrp marked this pull request as ready for review June 11, 2024 10:59
@FabianHofmann
Copy link
Contributor

yes, updating the repo's powerplants.csv sounds good

@lkstrp lkstrp merged commit d28290f into master Jun 11, 2024
2 checks passed
@lkstrp lkstrp deleted the build-gh-action branch June 11, 2024 11:38
@FabianHofmann
Copy link
Contributor

🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Create powerplants.csv and stats by GH action
3 participants