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

Added ACL 24 videos #3975

Closed
wants to merge 3 commits into from
Closed

Added ACL 24 videos #3975

wants to merge 3 commits into from

Conversation

davidstap
Copy link
Collaborator

No description provided.

@davidstap
Copy link
Collaborator Author

@mjpost Gentle ping on this PR

@mjpost mjpost added this to the 2024Q4 milestone Nov 7, 2024
@mjpost
Copy link
Member

mjpost commented Nov 7, 2024

@mjpost
Copy link
Member

mjpost commented Nov 7, 2024

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.

@davidstap
Copy link
Collaborator Author

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.

@mjpost
Copy link
Member

mjpost commented Nov 18, 2024

Thanks, @davidstap. I thought you used a heuristic mapping using titles and other information? Do they give you the mapping nowadays?

@davidstap
Copy link
Collaborator Author

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.

@davidstap
Copy link
Collaborator Author

Update: Underline responded; they are currently investigating the issue.

@mjpost mjpost assigned mjpost and unassigned mjpost and anthology-assist Nov 27, 2024
@mjpost
Copy link
Member

mjpost commented Dec 2, 2024

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?

@davidstap
Copy link
Collaborator Author

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.

@davidstap davidstap closed this Dec 2, 2024
@davidstap
Copy link
Collaborator Author

Still no updates, I have sent another reminder.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants