dmarx / video-killed-the-radio-star

Notebook and tools for end-to-end automation of music video production with generative AI
https://colab.research.google.com/github/dmarx/video-killed-the-radio-star/blob/main/Video_Killed_The_Radio_Star_Defusion.ipynb#scrollTo=oPbeyWtesAoh
MIT License
196 stars 35 forks source link

write_result() missing 1 required positional argument: 'options' #134

Closed mikeaschroth closed 1 year ago

mikeaschroth commented 1 year ago

Discussed in https://github.com/dmarx/video-killed-the-radio-star/discussions/130

Originally posted by **Kinggibbon1987** April 13, 2023 When I Infer speech from audio, I keep getting this error. Anyone know how to fix this ? This happens with and without whisper_seg checked ![image](https://user-images.githubusercontent.com/111515719/231812393-86387cd5-b663-4c92-be73-6eb9f95876d0.png)
NoLiesKDTV commented 1 year ago

hey man im stuck here too !

Maldus512 commented 1 year ago

The problem is due to this commit of the whisper dependency; since the notebook installs the library from the Github master it's vulnerable to breaking changes.

A workaround is simply to use a previous version either from a tag or a commit. The following change to the install section works:

!pip install git+https://github.com/openai/whisper@255887f219e6b632bc1a6aac1caf28eecfca1bac
# instead of this
# !pip install git+https://github.com/openai/whisper

Alternatively one should add this new option argument to the write_result call, but I'm not sure how it should work.