manim-voiceover icon indicating copy to clipboard operation
manim-voiceover copied to clipboard

Gradio

Open o-alexandre-felipe opened this issue 2 years ago • 6 comments
trafficstars

Add a recording service based on Gradio (https://gradio.app/) This enables one to record from the browser instead of the terminal. In particular it allows one to record when running manim on a different machine, e.g. via SSH, or in a docker container.

Also made some changes on the Recording service to record on a terminal without an X server.

o-alexandre-felipe avatar Apr 24 '23 19:04 o-alexandre-felipe

@o-alexandre-felipe Thanks for adding the new service. I will go through it shortly. In the meanwhile, can you check out why the build failed?

osolmaz avatar May 06 '23 10:05 osolmaz

Thank you @osolmaz ,

The failure was because I renamed the class MyListener to something less personal, but I forgot one reference.

o-alexandre-felipe avatar May 07 '23 05:05 o-alexandre-felipe

@osolmaz just wanted to give you a heads up, as I imagine this was lost in your inbox.

o-alexandre-felipe avatar May 19 '23 17:05 o-alexandre-felipe

Hey, sorry to keep you waiting, I was busy.

I've just built from your branch and am running into problems.

  • The interface of alternate RKeyListener is apparently not the same, it's giving me some errors (I've commented on the line that throws the error)
  • Running Gradio example, it launches locally and lets me record, but then hangs for minutes after I click 'accept'. Any idea why that happens?

Also made some changes on the Recording service to record on a terminal without an X server.

Can you elaborate this? Does pynput not work on Windows?

Also, once we get rid of all the errors and the PR is up to par, I would need GradioService to be added to the documentation, i.e. https://voiceover.manim.community/en/stable/services.html and the API reference.

Btw I just merged changes from the main branch, fyi.

Thanks!

osolmaz avatar May 20 '23 08:05 osolmaz

No problem, thank you for looking into it.

I will try to run the RecordingService again (which OS are you using?)

Not sure about why gradio would hang for so long. What platform are you using.

No, the existing code don't run in windows (because windows doesn't have X server), but the worst part for me is that I was trying to run in a docker image that don't have audio input.

So I will probably use this for my own installing from my github for now.

o-alexandre-felipe avatar May 22 '23 07:05 o-alexandre-felipe

I am on MacOS.

I also agree with you that there needs to be a web UI for this, for cross-platform compatibility. Gradio seems like a nice and quick solution, I'm super keen to get this merged once we iron the kinks out and get the API/docs in order :)

I will try to debug this when I have time soon, I am unfortunately unavailable during weekdays.

osolmaz avatar May 22 '23 14:05 osolmaz