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

Add citation cff file #330

Merged
merged 3 commits into from
Aug 3, 2021
Merged

Add citation cff file #330

merged 3 commits into from
Aug 3, 2021

Conversation

rgieseke
Copy link
Member

With GitHub adding support for CITATION.cff files to display a citation info on the side (see e.g. https://github.com/IAMconsortium/pyam/) i though it might be worth adding one here too.

https://twitter.com/natfriedman/status/1420122675813441540

With Zenodo also using this for displaying info on their website when dong releases i'm not sure how all of this will render out.

https://twitter.com/zotero/status/1420515377390530560

I've copied the citation info from the Readme and added a link to the 'latest' DOI from Zenodo.

We also ought to create releases to trigger a Zenodo archival, not sure if some beta versions should also be archived as they might have been used in papers?

Full spec: https://github.com/citation-file-format/citation-file-format

@rgieseke
Copy link
Member Author

One could potentially also have an update of the DOI field included in the CI automation.

@znicholls
Copy link
Collaborator

Nice. There's a chicken and egg issue with zotero (zotero reads the cff, but the cff can't be properly finalised until the new zotero DOI is minted, which requires the cff etc.). I don't know if new zotero DOIs are somehow predictable. Using latest the whole time is probably also ok.

We can probably make things tidier by using the reference key (https://github.com/citation-file-format/citation-file-format/blob/main/README.md#references-optional) rather than putting all the citations in the message.

This also allows a citation file to be generated from a zenodo entry: https://github.com/citation-file-format/doi2cff. I don't know if that helps with the chicken and egg problem though.

Maybe the workflow would be (although it's fiddly to automate)

  1. tag release
  2. reserve zenodo DOI (there are ways to automate this I think)
  3. update citation CFF with new zenodo DOI and version
  4. push to zenodo and publish release

https://github.com/openscm/openscm-zenodo should be able to handle all the zenodo stuff (might need an upgrade to allow automatic publishing though).

@rgieseke
Copy link
Member Author

Yeah, i looked into the 'references' field but couldn't find a tool quickly which would convert citations to cff format :-]
I was also unsure how GitHub would render these and the initial message.

Good point, on the DOI minting, i think it's maybe not worth the trouble ... Probably easiest to just use the 'latest' DOI or skip it altogether.
I think the main point of the Zenodo integration is that it uses it to read and format the title and authors (so no need for a zenodo.json file or manually editing those values on Zenodo.

@rgieseke
Copy link
Member Author

I'll add the references and keywords and then we can check how it looks?

@znicholls
Copy link
Collaborator

znicholls commented Jul 30, 2021 via email

@rgieseke
Copy link
Member Author

Presumably I'll have to specify the version tag in the cff for GitHub's widget to work. But Zenodo will get the git tag via the hook data, right? Feels a bit backwards.

Zenodo will use the version tag in the CFF instead of the hook data.

https://twitter.com/xrotwang/status/1420389754563145730

@znicholls
Copy link
Collaborator

Otherwise looks good

@znicholls znicholls mentioned this pull request Jul 31, 2021
4 tasks
@znicholls
Copy link
Collaborator

I think this is ready to go once #331 is merged and you rebase @rgieseke

@rgieseke rgieseke merged commit 34fc4c3 into master Aug 3, 2021
@rgieseke rgieseke deleted the add-citation-cff-file branch August 3, 2021 09:20
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.

None yet

2 participants