Skip to content

Conversation

@Azax4
Copy link
Collaborator

@Azax4 Azax4 commented Nov 14, 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).)

@Azax4 Azax4 added this to the Author page backlog milestone Nov 14, 2025
@Azax4 Azax4 self-assigned this Nov 14, 2025
@Azax4 Azax4 linked an issue Nov 14, 2025 that may be closed by this pull request
5 tasks
@github-actions
Copy link

Build successful. Some useful links:

This preview will be removed when the branch is merged.

- {first: Jeongwon, last: Cha}
- canonical: {first: V.G.Vinod, last: Vydiswaran}
variants:
- {first: VG Vinod, last: Vydiswaran}
Copy link
Contributor

Choose a reason for hiding this comment

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

Actually, I believe there are more papers where metadata and PDF content might not fully match. Some of this might be irrelevant for getting the paper on the same author page (so not sure which ones we need to list in the yaml explicitly, cf. https://github.com/acl-org/acl-anthology/wiki/Name-Variants ) , but it is nonetheless a slight inconsistency.

On the page with metadata displayed as "V.G.Vinod Vydiswaran" (without space in first name) I find papers with the following name copied from the PDF:

  • 2024.findings-emnlp.145 VG Vinod Vydiswaran (no punctuation, space added)
  • 2024.naacl-long.165 V.G. Vinod Vydiswaran (space added)
  • 2023.findings-acl.561 V.G. Vinod Vydiswaran (space added)
  • I17-1040 V. G. Vinod Vydiswaran (two! spaces added)
  • N10-4008 V.G. Vinod Vydiswaran (space added)

- canonical: {first: Jeong-Won, last: Cha}
variants:
- {first: Jeongwon, last: Cha}
- canonical: {first: V.G.Vinod, last: Vydiswaran}
Copy link
Contributor

@weissenh weissenh Nov 15, 2025

Choose a reason for hiding this comment

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

The issue submitter has provided an orcid. Why not add orcidand id field here?
Future papers can be mapped by orcid, issue submitter wasn't clear about it but I assume all papers currently listed on the page belong to the person, so we could verify the paper and person.

edit: and institution (or degree field) and comment field

Copy link
Contributor

@weissenh weissenh left a comment

Choose a reason for hiding this comment

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

I haven't done a full review yet, and will continue after the requested changes have been implemented (or argued why not necessary):

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: v-g-vinod-vydiswaran

3 participants