Skip to content

Improved authorship credits#612

Merged
brian-rose merged 5 commits intoProjectPythia:mainfrom
brian-rose:authors
Feb 24, 2026
Merged

Improved authorship credits#612
brian-rose merged 5 commits intoProjectPythia:mainfrom
brian-rose:authors

Conversation

@brian-rose
Copy link
Copy Markdown
Member

@brian-rose brian-rose commented Feb 24, 2026

This PR addresses #604 and does a few things related to authorship credits:

  • Maintains a full "source of truth" for Foundations authorship in the frontmatter of the How to Cite markdown
  • Lets MyST display the full author list in the rendered book, but only on the How to Cite page (link to the PR preview)
  • Replaces the old, manually maintained CITATION.cff file with one that can be dynamically created by MyST by running myst build preamble/how-to-cite.md

My thinking here is that the full author list to just too long to display on every page. However I want it to appear somewhere in the book, and the "How to Cite" page seemed like the most logical place for it.

Also, as I said over at #604, I am trying to reduce the amount of duplication of author metadata. The CITATION.cff file (which is used by GitHub and Zenodo) is now a reproducible product rather than a source of truth.

Future changes in the authorship list would be handled in the frontmatter of the one page where we display it. And the author data itself isn't defined in this repo at all, rather it is fetched from our upstream pythia.yml.

@github-actions
Copy link
Copy Markdown

github-actions bot commented Feb 24, 2026

👋 Thanks for opening this PR! The Cookbook will be automatically built with GitHub Actions. To see the status of your deployment, click below.
🔍 Git commit SHA: 40f8f72
✅ Deployment Preview URL: https://ProjectPythia.github.io/pythia-foundations/_preview/612

@brian-rose
Copy link
Copy Markdown
Member Author

Waiting on ProjectPythia/pythia-config#21 before the full author list looks correct here.

@brian-rose
Copy link
Copy Markdown
Member Author

This PR also adds a heads-up statement about our JOSE paper while we wait for a DOI to be issued (#605).

@brian-rose
Copy link
Copy Markdown
Member Author

Also, this PR is essentially a workaround for the lack of support in MyST for controlling how (or whether) the author list is displayed at the top of each rendered page.

This may well change in the future. Tracking related MyST feature issues jupyter-book/mystmd#1664 and jupyter-book/mystmd#2394.

If it becomes possible to do things like display all authors on each page but put them in a discreet dropdown box, then we can move the full author list back to the book-level myst.yml.

@brian-rose brian-rose marked this pull request as ready for review February 24, 2026 16:38
@brian-rose brian-rose requested a review from a team as a code owner February 24, 2026 16:38
@brian-rose brian-rose requested review from clyne and dcamron and removed request for a team February 24, 2026 16:38
@dcamron
Copy link
Copy Markdown
Contributor

dcamron commented Feb 24, 2026

Neat changes, thanks for the helpful context

Copy link
Copy Markdown
Contributor

@kafitzgerald kafitzgerald left a comment

Choose a reason for hiding this comment

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

Thanks, this looks good to me overall (especially given the current constraints).

I did see a blank entry that got inserted (see comment). Not sure if this is intended or something that might be easy to drop.

@dcamron dcamron self-requested a review February 24, 2026 18:36
@brian-rose
Copy link
Copy Markdown
Member Author

Thanks folks! Merging now.

@brian-rose brian-rose merged commit 0cf8491 into ProjectPythia:main Feb 24, 2026
3 checks passed
@brian-rose brian-rose deleted the authors branch February 24, 2026 20:44
github-actions bot pushed a commit that referenced this pull request Feb 24, 2026
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