Skip to content

feat: add contributors to upstream #12

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

Merged

Conversation

agoose77
Copy link
Contributor

@agoose77 agoose77 commented Jun 19, 2025

Use the contributors field so that downstream we can refer to authors by their GitHub IDs.

These were taken from blogs, and should be extended as needed with new authors and metadata.

@jukent @ktyle @clyne @brian-rose would you mind confirming that your ORCIDs are correct, and add any other metadata you're comfortable aggregating here. It will show up e.g. on blog posts that you're added to as an author or contributor.

These were taken from blogs, and should be extended as needed
@brian-rose
Copy link
Member

brian-rose commented Jun 19, 2025

Hi @agoose77, this looks great!

We have a pretty detailed set of author metadata covering many past and present Pythia contributors in the CITATION.cff file for Pythia Foundations.

Would it be useful to include the website: and affiliation: fields that are included in that citation file here?

@agoose77
Copy link
Contributor Author

@brian-rose yes, as much information as you can handle. However, I think it would be really nice if these two had a single source of truth. I know that we can generate a cff export -- did you run into problems with that when you tried it?

@brian-rose
Copy link
Member

I'm not sure. I know that we settled on the CITATION.cff file a couple years back as the version of record for authorship for each repo because it is digested by both GitHub and Zenodo.

@brian-rose
Copy link
Member

However, I think it would be really nice if these two had a single source of truth.

+1 for this! There is a lot of repetition across the cookbook repos since many of the same individuals are attributed as authors across multiple works.

Copy link
Member

@brian-rose brian-rose left a comment

Choose a reason for hiding this comment

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

Suggestions here to add a few more fields that we track in our various CITATION.cff files.

@brian-rose
Copy link
Member

I think it makes sense to go ahead with this now as a "proof of concept" for the blog posts in ProjectPythia/projectpythia.github.io#526.

I have some thoughts and questions about the larger question of maintaining a single version of authorship data that can be reused in various ways across our many repos, including automated CITATION.cff generation. But none of that needs to be resolved before merging this and trying it out on the blog.

@brian-rose
Copy link
Member

@brian-rose brian-rose merged commit f2b2412 into ProjectPythia:main Jun 20, 2025
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