Skip to content

fix orcidlink for revtex#2

Merged
misharash merged 2 commits intomainfrom
revtex-orcid-fix
Mar 9, 2026
Merged

fix orcidlink for revtex#2
misharash merged 2 commits intomainfrom
revtex-orcid-fix

Conversation

@p-slash
Copy link
Copy Markdown

@p-slash p-slash commented Feb 17, 2026

This fixed the ORCID links for PRD for me. I am using Revtex4-2 in Overleaf.
In short: The current output, \author[ORCID]{John Doe}, does not work. This change instead outputs \author{John Doe\orcidlink{ORCID}}.

@misharash
Copy link
Copy Markdown

misharash commented Mar 6, 2026

Hi Naim, sorry I missed this (turned out I was not subscribed to mkauthlist GitHub updates despite getting them from some other cosmodesi repos I am not related to).
Did you mean that the current output does not render the ORCID links (indeed, it does not seem to)? Or does it cause compilation errors for you?
On the other hand, some journals may be using the current format for parsing author information; we might want to double-check this.
(The failing check/test is a hard-coded comparison; it can be fixed easily after we figure out the more conceptual issues.)

@misharash
Copy link
Copy Markdown

I thought this would at least affect @paulmartini's mkauthorxml script, but it actually looks like it won't.

@misharash
Copy link
Copy Markdown

misharash commented Mar 6, 2026

@misharash
Copy link
Copy Markdown

misharash commented Mar 9, 2026

Actually, the ORCID format support was introduced in AASTeX 6.1: https://journals.aas.org/aastex-v6-1-author-guide/ (not mentioned for 6.0 https://journals.aas.org/aastex-v6-0-author-guide/), but still not in RevTeX it seems. So I think the right thing would be to keep the old format for aastex6, but use Naim's change for revtex.

@misharash misharash merged commit c936444 into main Mar 9, 2026
6 checks passed
@misharash
Copy link
Copy Markdown

Thank you Naim, and sorry for possibly excessive scrutiny — although it did uncover a subtlety with the affected aastex6 case.

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