Skip to content

Conversation

@weissenh
Copy link
Contributor

@weissenh weissenh commented Nov 16, 2025

(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).)

Closes #6131

TL;DR: two author pages with a total of 121 papers should merge into one based on request by the author. The merge was necessary due to different verbosity in first name (name variants) and wrongly assigning middle name to last name sometimes. Degree institution and ORCID have been added too. A few of the papers required metadata corrections on the way (on his name or on co-authors)

TODO:

  • check again in preview
  • name variants that are no longer needed?

Verification / Sources

Author pages

(paper numbers by searching for 'bib' on page)

Observations

Decided on canonical name as "John Philip McCrae": that's also the name displayed on his website, preferred one on OpenReview and Anthology URL listed there, the URL with the respective slug contained the majority of papers anyway, and it is more elaborate than others (John P./John).

also correcting likely mistakes (Philip to last name, non-capitalized C)
Still need to check metadata matches PDF
@weissenh weissenh added this to the Author page backlog milestone Nov 16, 2025
@weissenh weissenh self-assigned this Nov 16, 2025
@weissenh weissenh linked an issue Nov 16, 2025 that may be closed by this pull request
5 tasks
- years 2013-2016, 2024, 2025
- as a side note also added missing diacritics, corrected author order, added missing first name part...
- year 2025 (this time  for real)
- as a side note also added missing diacritics, added missing first name part...
- year 2018
- as a side note also added missing first name part...
- year 2019
- as a side note also changed name order for one co-author
- year 2023
- as a side note also added diacritis to one co-author and another first name
- year 2022
- as a side note also changed co-author names and order to match PDF content
- year 2021, 2020
- as a side note also changed co-author names
@mjpost
Copy link
Member

mjpost commented Dec 13, 2025

@weissenh is this still in draft? What remains to be done?

@weissenh
Copy link
Contributor Author

is this still in draft? What remains to be done?

@mjpost I've taken notes on what is left to be done in the description of the PR

TODO:

  • check again in preview
  • name variants that are no longer needed?

To elaborate I'd planned to check that all papers in the preview are now on the correct page (total paper number matches) and I was wondering which name variants needed to be listed explicitly, some might be inferred by our matching logic and fixing metadata-PDF inconsistencies I guess there are variants which are no longer needed?

middle name is now consistently attached to first name, not last name
@weissenh weissenh marked this pull request as ready for review December 17, 2025 01:03
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.

Author Page: john-p-mccrae

3 participants