Skip to content

Handling no speech video#62

Merged
zh-plus merged 1 commit intozh-plus:masterfrom
MaleicAcid:master
Dec 12, 2024
Merged

Handling no speech video#62
zh-plus merged 1 commit intozh-plus:masterfrom
MaleicAcid:master

Conversation

@MaleicAcid
Copy link
Copy Markdown
Collaborator

@MaleicAcid MaleicAcid commented Dec 11, 2024

Create default subtitle file instead of raise exception.

@MaleicAcid MaleicAcid closed this Dec 11, 2024
@MaleicAcid MaleicAcid reopened this Dec 11, 2024
@MaleicAcid
Copy link
Copy Markdown
Collaborator Author

MaleicAcid commented Dec 12, 2024

It seems that errors occurred when calling the openai api. Is the openai token used in ci/cd available?
The no-speech test can run successfully on my dev machine(use openai forward site).

python -m unittest test_openlrc.TestLRCer.test_nospeech_video_file_transcription_translation

@zh-plus zh-plus self-requested a review December 12, 2024 12:26
@zh-plus zh-plus merged commit 52a3c41 into zh-plus:master Dec 12, 2024
@zh-plus
Copy link
Copy Markdown
Owner

zh-plus commented Dec 12, 2024

Thanks for your PR!

@zh-plus zh-plus removed their request for review December 12, 2024 12:56
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