-
Notifications
You must be signed in to change notification settings - Fork 30
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 es, eu & fa #88
base: master
Are you sure you want to change the base?
add es, eu & fa #88
Conversation
Why are we adding languages when the clips file was frozen several weeks ago? |
Oh I can get you a new clips file, it's part of the new export. But part of the export is also running CorporaCreator, which currently causes the export to fail :/ |
Getting a new clips is not the problem. I'm more commenting on the fact that we should have, and as far as I understood we did, freeze the release clips.tsv weeks ago to focus on cleaning it up instead of adding new languages. |
Oh, I didn't think those were contradictory goals, as cleaning up is an ongoing effort. |
My point is that we have to, and my understanding is that we did several weeks ago, freeze clips.tsv Adding newer and newer clips.tsv files closer and closer to the deadline will only ensure that the data set is not sufficiently vetted. |
Okay, I'll reach out to the rest of the team to make sure we're aligned on this. |
Did you get any answers from the rest of the team? |
Not really, I posted it in our internal Slack channel (so that you would see it too). I just went with the assumption that your memory is better than mine and changed the bundling to only get clips from the same time range as the clips.tsv dump. |
No description provided.