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

Name update for Toby Jia-Jun Li #4669

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

tobyli
Copy link

@tobyli tobyli commented Feb 19, 2025

merge the author pages of "Toby Li" and "Toby Jia-Jun Li"

(Please replace this text with a description of the changes effected by this pull request.
Include a link to the corresponding Github Issue, if there is one.
Details on how to do this (can be found here).)

merge the author pages of "Toby Li" and "Toby Jia-Jun Li"
@mjpost
Copy link
Member

mjpost commented Feb 19, 2025

Hello, we can merge this, but please note a few things:

  • We view PDFs as authoritative. The metadata on your papers doesn't match, and should be corrected for each of them to "Toby Jia-Jun Li"
  • If you want a different canonical name for your author page, that is still a matter of editing fie file you are editing, but you will have to update it to anticipate these corrections
  • Please update your name in Open Review / Softconf so that metadata will be correct moving forward. Actually, it may be correct; for a while, aclpub2 was dropping middle names, which is maybe what happened to you.

@mjpost mjpost self-assigned this Feb 19, 2025
@mjpost mjpost changed the title Update name_variants.yaml Name update for Toby Jia-Jun Li Feb 19, 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