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

[Bug?] Problems with UVR-DeEcho-DeReverb #4

Open
ShiromiyaG opened this issue Apr 1, 2024 · 4 comments
Open

[Bug?] Problems with UVR-DeEcho-DeReverb #4

ShiromiyaG opened this issue Apr 1, 2024 · 4 comments

Comments

@ShiromiyaG
Copy link

ShiromiyaG commented Apr 1, 2024

I tested several models, both MDX, MDXC and VR, but the only model that I had results that were actually worse than the UVR GUI was the UVR-DeEcho-DeReverb. Here is a link to a song comparing UVR-DeEcho-DeReverb and MDX Reverb HQ from the UVR api, along with UVR-DeEcho-DeReverb from the UVR GUI

https://drive.google.com/drive/folders/1_E0PzZRtK4f8Ah3rknJBDitZZvDNnFeO?usp=drive_link

And this is the code I used:

MDX = models.MDX(name="Reverb_HQ",  other_metadata={'segment_size': 256,'overlap': 0.75,'mdx_batch_size': 8,'semitone_shift': 0,'adjust': 1.08, 'denoise': False,'is_invert_spec': False,'is_match_frequency_pitch': True,'overlap_mdx': None},device=device, logger=None)
res = MDX(name)
seperted_audio = res["no reverb"]
Vr = models.VrNetwork(name="UVR-DeEcho-DeReverb", other_metadata={'normaliz': False,'aggressiveness': 0.05,'wav_type_set': "PCM_F32",'window_size': 320,'batch_size': 4,'is_tta': True},device=device, logger=None)
res = Vr(name)
seperted_audio = res["no reverb"]

Also, a photo of Audacity comparing Reverb HQ and UVR-DeEcho-DeReverb:
image

@MohannadEhabBarakat
Copy link
Contributor

Thanks 🤩. I'll check your results and try to figure out why this happened. I'm just curious, did you use any custom parameters in UVR?

@ShiromiyaG
Copy link
Author

The only thing I changed was the window size, which I set to "320" and the aggression setting, which I set to "5", otherwise everything is in the default setting.

@MohannadEhabBarakat
Copy link
Contributor

Thats interesting. The code looks good. And I believe that UVR doesn't use secondary models by default, which means that the results should be identical. I'll check this later around 12th when I come back

@ShiromiyaG
Copy link
Author

Yes, UVR does not use secondary models by default, and in this case, I did not activate it

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