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

Add missing ACL 24 papers #3753

Merged
merged 6 commits into from
Aug 27, 2024
Merged

Add missing ACL 24 papers #3753

merged 6 commits into from
Aug 27, 2024

Conversation

mjpost
Copy link
Member

@mjpost mjpost commented Aug 14, 2024

Many papers were missed in the ingestion. This adds

  • Short papers
  • Long papers

Copy link

Build successful. Some useful links:

This preview will be removed when the branch is merged.

@akoehn
Copy link
Member

akoehn commented Aug 16, 2024

Why did all these hashes change? Was the initial data changed for all these papers at some point in time?

@mjpost
Copy link
Member Author

mjpost commented Aug 16, 2024

The volume title was changed (to add "Volume 2: Short Papers"), so all the footers changed.

Copy link
Member

@akoehn akoehn left a comment

Choose a reason for hiding this comment

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

I did not check the checksums but I assume they are correct

@TheShayegh
Copy link

Here is a list I prepared: https://theshayegh.github.io/ACL2024MissingPapers/

@mjpost
Copy link
Member Author

mjpost commented Aug 22, 2024

@TheShayegh thanks. ACL is still working on this actively and has the list in OR. I hope we will have this resolved soon.

@mjpost mjpost merged commit a126203 into master Aug 27, 2024
2 checks passed
@mjpost mjpost deleted the acl-24-corrections branch August 27, 2024 17:01
xinru1414 pushed a commit that referenced this pull request Aug 29, 2024
* Add missing ACL 24 short papers
* Add missing ACL long papers
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