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

Gradio #44

Open
wants to merge 8 commits into
base: main
Choose a base branch
from
Open

Conversation

o-alexandre-felipe
Copy link

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.

@osolmaz
Copy link
Collaborator

osolmaz commented May 6, 2023

@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?

@o-alexandre-felipe
Copy link
Author

Thank you @osolmaz ,

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

@o-alexandre-felipe
Copy link
Author

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

@osolmaz
Copy link
Collaborator

osolmaz commented May 20, 2023

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!

@o-alexandre-felipe
Copy link
Author

o-alexandre-felipe commented May 22, 2023

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.

@osolmaz
Copy link
Collaborator

osolmaz commented May 22, 2023

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.

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.

None yet

2 participants