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

Perform speech recognition if dst_language is not denied. #87

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

nvh95
Copy link

@nvh95 nvh95 commented Apr 6, 2018

Currently, if user provides src_language and don't provide dst_language argument, autosub still performs Speech Recognition, but then return an error (Subtitle translation requires specified Google Translate API key.) and the generated subtitle is not saved. (because src_language and dst_language are not the same)
screen shot 2018-04-06 at 11 14 47 am

In this pull request, if user provides only src_language, autosub will perform Speech Recognition and saved the subtitle.

nvh95 added 2 commits April 6, 2018 10:41
…if anyone use `generate_subtitles` function, they don't have to pass dst_language as a required parameter.
@agermanidis agermanidis force-pushed the master branch 3 times, most recently from 20e218f to 477d5a9 Compare October 10, 2018 17:18
@BingLingGroup
Copy link

If args.dst_language is None, just let args.dst_language = arg.src_language and that would be simpler than setting a is_dst_language_provided variable.

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.

2 participants