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

Fix Author List on Zenodo #1011

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from
Open

Conversation

emanuel-schmid
Copy link
Collaborator

@emanuel-schmid emanuel-schmid commented Feb 12, 2025

Changes proposed in this PR:

  • Without .zenodo.json file in the root directory of a repository, the automated creation of a release doi derives all information from the repository itself, Consequently authors are contributors as listed in the settings of the project. In order to control the listed authors and affiliations this PR introduces a .zenodo.json file which overwrites the respective attributes.

This PR fixes the issue with authors like 'climada-jenkins' listed in Zenodo.

PR Author Checklist

PR Reviewer Checklist

@emanuel-schmid
Copy link
Collaborator Author

weather the doi is then created as we wish still needs to be seen. I've made some tests on a fake repository of mine though and I'm +/- confident.

But as for the correct affiliation and the order of authors and whether they should be listed at all, I'm not so sure.
The way it is now I tried to list all authors that appear at all in the insights and have a minimum number of changed lines of ~20.
The affiliations were then taken from what the authors assigned in GitHub.

@davidnbresch @chahank @peanutfun please have a look

@chahank
Copy link
Member

chahank commented Feb 12, 2025

Curious to see how stable this is. I played around with this for the climada papers in the past, but did not manage to get it to work.

I would change the affiliations to the correct ones though. (e.g., Zelie is '5' ?).

@emanuel-schmid
Copy link
Collaborator Author

Curious to see how stable this is. I played around with this for the climada papers in the past, but did not manage to get it to work.

me too. 😁 currently I'm hoping that deleting the zenodo webhooks made a difference. At least the new one claims to have been successful. Although on Zenodo nothing has changed. We shall see ...

I would change the affiliations to the correct ones though. (e.g., Zelie is '5' ?).

👍 are there others? Even if the automated upload doesn't work we still need this list to be correct, because plan B is using the zenodo-api to do the same, but with a github action script.

@chahank
Copy link
Member

chahank commented Feb 13, 2025

I would say about 50% of the affiliations are currently wrong, and another few are incomplete ^^. Should I manually edit the file?

@emanuel-schmid
Copy link
Collaborator Author

Listing the affiliations here or sending me an update by mail or slack is just as fine.

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.

2 participants