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

Update 2024.eacl.xml for the paper with anthology ID: 2024.eacl-long.117 #3261

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

Conversation

ipavlopoulos
Copy link

The software does not exist and the link is broken. The software should not exist, because the paper shares a GitHub repository. But a video should exist.

Removed the link to the attachment.

The software does not exist, the paper shares a GitHub repository. Removed. A video should exist however.
@ipavlopoulos ipavlopoulos changed the title Update 2024.eacl.xml for paper with ID 115 Update 2024.eacl.xml for the paper with anthology ID: 2024.eacl-long.117 Apr 24, 2024
@mjpost
Copy link
Member

mjpost commented May 1, 2024

It seems that no attachments exist for EACL 2024. @anthology-assist, can you please check if these were inadvertently not uploaded?

@anthology-assist
Copy link
Collaborator

Looks like the attachments were not uploaded. But it also seems like some attachments shouldn't be uploaded (like this PR mentioned). Shall we remove all attachment element for EACL?

@mjpost
Copy link
Member

mjpost commented May 17, 2024

Authors are free to upload data when they submit their papers, independent of whether they have a github link in their paper. In fact doing so serves an archival purpose since the Github link may disappear. But it's worrisome that we created the link to the attachment when it was not in fact there. That shouldn't be possible from the ingestion script.

I would also check with the EACL pub chairs to see if these submissions were inadvertently lost.

@mjpost
Copy link
Member

mjpost commented May 17, 2024

Videos will be added soon in #3296.

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.

3 participants