-
Notifications
You must be signed in to change notification settings - Fork 0
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
Enforce selection of media file language #71
Conversation
Co-Authored-By: Marjan Kalanaki <15894063+marjisound@users.noreply.github.com>
Co-Authored-By: Marjan Kalanaki <15894063+marjisound@users.noreply.github.com>
Co-Authored-By: Marjan Kalanaki <15894063+marjisound@users.noreply.github.com> Co-Authored-By: Sam Cutler <5560113+itsibitzi@users.noreply.github.com> Co-Authored-By: Luke Hoyland <luke.hoyland@theguardian.com>
this isn't true - when we pass language 'auto', the model detects the language that most of the audio is in, then only transcribes audio in that language, ignoring the rest
// being submitted without any files selected. Need to confirm this in | ||
// order to narrow the type of files | ||
if (files === null || files.length === 0) { | ||
console.error('form submitted without any files'); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
since we don't have anything shipping client side logs anywhere, I'm not sure it's worth logging here
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks like a well thought out improvement, makes sense!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🚢
Co-Authored-By: Sam Cutler <5560113+itsibitzi@users.noreply.github.com> Co-Authored-By: Luke Hoyland <luke.hoyland@theguardian.com>
What does this change?
Dropdown
element to a reactSelect
element. The former didn't work on some mobile devices. When it was selected, the dropdown opened upward (regardless ofplacement
property) and couldn't be navigated by swiping or scrolling.useState
hookBefore
auto-detect language is the default, language picker doesn't work
language.picker.bug.mov
after
no default language, language picker does work
language.picker.fixed.mov
How to test
How can we measure success?