-
Notifications
You must be signed in to change notification settings - Fork 317
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
Added ACL 24 videos #3975
Added ACL 24 videos #3975
Conversation
Build successful. Some useful links:
This preview will be removed when the branch is merged. |
@mjpost Gentle ping on this PR |
I did a bunch of spot-checking and found mistaken alignments on these papers, which suggests there are likely many more: |
Another thing we should be thinking about is how this process could be improved. At some point in the video creation process, there is a definitive alignment of videos to papers. It would be nice to be able to make use of that instead of having to rely on semi-automatic post-hoc heuristic alignment. |
Thanks for catching that - it turns out my random sample happened to be lucky. I investigated and found that the file from Underline has mismatched titles and videos. (This was the source of earlier errors too.) I'll contact them about fixing it, though they're likely to be tied up with EMNLP next week. |
Thanks, @davidstap. I thought you used a heuristic mapping using titles and other information? Do they give you the mapping nowadays? |
Yes, Underline provides the mapping nowadays. Given the mismatches I assume they do this heuristically. I haven't got a response from them yet, will send a reminder. I'll also discuss how to improve the process to prevent these errors in the future. |
Update: Underline responded; they are currently investigating the issue. |
Any updates? Might be worth pinging them again. I also see there are lots of conflicts; can you either resolve them, or maybe it's easier to just start over once you have the updates? |
No updates, I'll send another reminder. I think it's easier to simply start over given all the conflicts, I'll close this PR and open a new one when I receive the updated data from Underline. |
Still no updates, I have sent another reminder. |
No description provided.