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

local Pinokio's version is broken #3

Open
eyevenear opened this issue Mar 12, 2024 · 3 comments
Open

local Pinokio's version is broken #3

eyevenear opened this issue Mar 12, 2024 · 3 comments

Comments

@eyevenear
Copy link

when I import a custom audio to transform in a Pinokio installation, it spits this error:

" ValueError: File S:\ai_apps\PINOKIO\cache\GRADIO_TEMP_DIR\a4f7b8bae81d9dfd80367d2fdbc18394a0cad45f\2.mp3 is not in the upload folder and cannot be accessed. "

please tell me what i need to do to make this work, thank you for this app is a dream come true!

@HilaManor
Copy link
Owner

Hi!
I have no idea how Pinokio works, however I suspect it is related to changes I pushed today to the hugging face space (The installation probably takes the most recent commit).

Specifically, commit a0738ba makes it so intermediate variables (wts, zs in the code) are not saved in the GPU memory anymore, but rather on disk (in Gradio's cache dir), and changed it so they are deleted every hour.
I had to do that to prevent memory leaks since Gradio doesn't automatically frees variables once a user leaves the site. (The variables cut run-time in some configurations)

I'm planning in the weekend to check another way of handling the memory leak, and if it works it might solve the Pinokio problem. In the meanwhile, can you maybe revert the code in the installation to the commit before that? Or even better, change the code (because I also pushed later another commit that fixes some edge case regarding timesteps)?

If yes, the fix depends on the problem:

  1. From the error maybe the cache dir was not located succesfully, and so you need to make sure it writes to and reads from the correct place.
  2. Maybe the error is that the file was just deleted because an hour passed, so maybe remove the empty_cache argument from the gr.Blocks definition.

I hope this helps

@eyevenear
Copy link
Author

i use a 4090, would it be possible for you to release an installer with a local gui?

Thank you so much

@HilaManor
Copy link
Owner

Hi, I do not know how to release an installer for the hugging-face GUI.
The pinokio app I think you're using was not made by me, but rather @cocktailpeanut (See this twitter post).
I still couldn't find another way of handling the memory leak, so unless I find one @cocktailpeanut will probably know how to help with that regard.

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

No branches or pull requests

2 participants