Videos are copied twice to RAM about facefusion

[henryruhs]

Like I know the Gradio hash... anyway, I gonna fix that asap. Thanks for the report.

[henryruhs]

Could you help me understanding the issue by providing some screenshots? I doubt it is loaded 2 times.

[felixsanz]

Sure. I deleted /tmp/gradio to make sure nothing exists.

Then I drag and drop the video into the target zone.

And if you check /tmp/gradio now...

2 folders with different hash but the same video. Since this is mounted to RAM it's always consuming double RAM than needed.

I know I can move the temp folder to a disk but in RAM is fast and nice, it's just that it shouldn't be here duplicated.

[henryruhs]

This is indeed an issue cause by the my integration.

To have just one drop area for images and videos I use gradio.File instead of gradio.Image and gradio.Video next to each other. According to the media type I display gradio.File instead of gradio.Image.

I didn't expect they load that file twice to the RAM.

[felixsanz]

It happens with every asset. The face image you upload to the "source" zone, it happens too (if that helps)

[henryruhs]

The source.py is made the same way like the target.py - not because it is needed but to have the same UI concept.

[henryruhs]

Could be solved either with Tabs or by unloaded the File components once the media type is known.

[felixsanz]

Unloading the old "file" instance and keeping the "video" or "image" one seems to be a better solution to me, so the UI doesn't get more complex without needing

[henryruhs]

I agree... but that needs lot of refactoring

[felixsanz]

I agree... but that needs lot of refactoring

why? just when the file is uploaded... there is already 2 times, remove the old one with os.remove()

[henryruhs]

The good news: source_file and target_file are no longer used across the application. I use source_image, target_image and target_video now.

The bad news: unloading the file components does not result in the wanted behavior.

This might stay unfixed for 1.1.0 in case nobody sends a pull request until tomorrow.

[henryruhs]

I created a issue: gradio-app/gradio#5809

Please vote it up!

[henryruhs]

I resolved this as wontfix... closed on the Gradio side - the decision is made that we move away from this framework.