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

Create Zenodo DOI #114

Closed
bernhardmgruber opened this issue Oct 28, 2020 · 6 comments · Fixed by #292
Closed

Create Zenodo DOI #114

bernhardmgruber opened this issue Oct 28, 2020 · 6 comments · Fixed by #292

Comments

@bernhardmgruber
Copy link
Member

We should have each release uploaded to Zenodo and a DOI created for it.

@bernhardmgruber
Copy link
Member Author

I cannot connect Zenodo to the Github LLAMA repository, because I do not have the rights to allow Zenodo to access the repository.

@psychocoderHPC, @BenjaminW3 or @ax3l: you are listed as admins of the alpaka-group. Could someone of you please grant access to Zenodo? Here is a guide: https://docs.github.com/en/free-pro-team@latest/github/setting-up-and-managing-organizations-and-teams/approving-oauth-apps-for-your-organization
You might not see a Zenodo request, because I cannot create one from my Zenodo account :/

Thank you!

@bernhardmgruber
Copy link
Member Author

The Github integration on Zenodo is now enabled. However, no DOI has been created yet for any of the previous releases.

@psychocoderHPC
Copy link
Member

If you go to the release page than edit the last commit and store (no changes) the event should be fired again. If you have a DOI please add this DOI to the release nodes

@bernhardmgruber
Copy link
Member Author

The Zenodo hook was triggered during the last release, but there were errors on the Zenodo side:

{
    "errors": "{'metadata': {'grants': [u'Invalid grant ID(s): 10.13039/501100000780::05E18CHA'], 'contributors': {0: {'type': [u'Missing data for required field.']}, 1: {'type': [u'Missing data for required field.']}, 2: {'type': [u'Missing data for required field.']}}}}"
}

@ax3l
Copy link
Member

ax3l commented Jun 8, 2021

The errors are on our side, I fear.

The contributors need a type field, e.g.:

llama/.zenodo.json

Lines 21 to 34 in c60373b

"contributors": [
{
"name": "Wesarg, Bert",
"affiliation": "TU Dresden"
},
{
"name": "Velten, Markus",
"affiliation": "TU Dresden"
},
{
"name": "Werner, Matthias",
"affiliation": "Helmholtz-Zentrum Dresden-Rossendorf"
}
],

vs.
https://github.com/openPMD/openPMD-api/blob/34b24d92f262590f3da2e5d587e5904640fd76d8/.rodare.json#L29

And one of the grant ID is invalid:
https://github.com/alpaka-group/llama/blob/develop/.zenodo.json#L50

Maybe remove the grants, add them for the first release manually, then export the JSON file and check their correct IDs for the .zenodo.json file.

@bernhardmgruber
Copy link
Member Author

Thank you @ax3l! That is really helpful! I will give your suggestions a try!

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 a pull request may close this issue.

3 participants