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

Adds a a new file CITATION.cff #5721

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

ljhwang
Copy link
Contributor

@ljhwang ljhwang commented Jun 1, 2024

The use of CITATION.cff files is becoming more widely accepted. It is supported by GItHub, Zenodo and Zotero. In addition, projects that do metadata analysis on software often scrape open repositories for this information. I am recommending we include this file so it can be included in such studies.

Zenodo will in the near future transition to CITATION.cff away from .zenodo.json. The .cff file should be considered a substitute and the .json file deleted.

CAUTION:

  • I believe this file overrides defaults when using the Zenodo webhook which I do not think we use anyways.
  • This file contains both the concept DOI and the version DOI. Hence, we this would need to be updated in new releases.

Named principle developers should review their personal information in this file.

For more information, see: https://citation-file-format.github.io/

Thoughts?

CITATION.cff Outdated Show resolved Hide resolved
CITATION.cff Outdated Show resolved Hide resolved
CITATION.cff Outdated Show resolved Hide resolved
CITATION.cff Outdated Show resolved Hide resolved
CITATION.cff Outdated Show resolved Hide resolved
CITATION.cff Outdated
Comment on lines 62 to 68
ASPECT is a code to simulate convection and tectonic
processes in Earth and other planetary bodies. It has
grown from a pure mantle-convection code into a tool for
many geodynamic applications including applications for
inner core convection, lithospheric scale deformation,
two-phase flow, and numerical methods development. The
project is supported by CIG (https://geodynamics.org).
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe it's worth spelling out what ASPECT stands for? I don't like documenting history, I generally like it better if we document the current state. So I would rewrite the "It has grown from..." part. I feel not very strongly about this here, though.

What do others think?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good point. I modified this. Additional wordsmithing welcome.

CITATION.cff Outdated Show resolved Hide resolved
Copy link
Member

@tjhei tjhei left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We would need to add a todo item to the release task list to update it.

CITATION.cff Outdated Show resolved Hide resolved
Comment on lines +53 to +58
- type: doi
value: 10.5281/zenodo.592692
description: concept DOI
- type: doi
value: 10.5281/zenodo.8200213
description: version 2.5.0
Copy link
Member

@tjhei tjhei Jun 2, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't quite understand the purpose of this section. What is "concept DOI"? Both DOIs link to ASPECT 2.5.0 on Zenodo, right? What's with this description: shouldn't that be the name of the article (ASPECT 2.5.0)? Does this prefill anything on zenodo when we make a new release?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A concept doi is a link to the project (the concept of the idea if that makes any sense) as a whole and not to a specific version.

@tjhei
Copy link
Member

tjhei commented Jun 2, 2024

  • Zenodo webhook which I do not think we use anyways.

We use the Zenodo webhook. Can you delete the zenodo.json file as well? (this assumes it is already supported by zenodo)

@tjhei
Copy link
Member

tjhei commented Jun 2, 2024

If this file contains the version number of the latest release and the DOI, how can one make a software release, generate a Zenodo DOI automatically that does not contain outdated information in this file? (the old json file does not contain the version number, just a list of people)
@ljhwang Maybe you can explain to us later how this is supposed to work.

@ljhwang
Copy link
Contributor Author

ljhwang commented Jun 2, 2024

@tjhei
So currently there is little impact on adding a CITATION.cff file. The .cff file is used by GitHub to generate the citation in the repository - Cite this repository drop down. See https://docs.github.com/en/repositories/managing-your-repositorys-settings-and-features/customizing-your-repository/about-citation-files.

However this may not be the functionality we seek. The current drop down points to the webpage.

In addition, if you have a .zenodo.json file in your repo, the .json file will override the .cff file. See the FAQa under GItHub: https://help.zenodo.org/faq/

Conclusion is that probably this is premature. It will allow the project to be scraped but will impact how the citation is displayed in the repo. Perhaps it is better to wait for the codemeta.json integration which my source confirms it has not made it into production. I did not get an estimated timeline and this does not show up on their roadmap unless it is a small piece of a larger project e.g. EU Open Research Repository.

Shrug

Co-authored-by: Timo Heister <[email protected]>
@bangerth
Copy link
Contributor

bangerth commented Jun 2, 2024

Independent of the question of whether this particular file is useful, my suggestion would be to use the list of principal developers, not maintainers. The former is more inclusive and in particular gives @cedrict credit for the many cookbooks he has written over the years.

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.

3 participants