Skip to content

Conversation

@hyperuser178
Copy link

Functions wavread & wavread_rosa are added, where the latter is called as ffread_rosa in the file rvc/modules/uvr5/vr.py as an alternative, in order to avoid bugs that librosa(.core).load may cause.

Due to that the param res_type is defined in librosa, the resampling is also performed by librosa.resample in the function(s).

However, due to that I'm not sure what kind of unit tests are required, it hasn't been performed / passed unit tests.

This function reads audio files using ffmpeg, allowing for asynchronous reading and various output configurations.
…mpling

Updated `wavread` function to include resampling using `librosa`, & added `wavread_rosa` for `librosa`-like params.

Due to that the param `res_type` is defined in `librosa`, resampling is performed using `librosa.resample` as well.
Added option to load audio using ffmpeg in process method.
The function is implemented in new file `.../uvr5_pack/ffio.py`.
However, it hasn't been performed serious unit tests yet due to that I'm not sure what kind of unit tests are required.
@hyperuser178
Copy link
Author

新增函数wavreadwavread_rosa,其中后者在文件rvc/modules/uvr5/vr.py中作为替代方案被命名为ffread_rosa,以规避librosa(.core).load可能引发的错误。

由于参数 res_typelibrosa 中定义,重采样操作在函数中也通过 librosa.resample 实现。

但由于不确定需要何种单元测试,目前尚未进行/通过单元测试。


我是在读 UVR5 实现时发现的或许可以改动的此处,但还没做单元测试,其实不仅是因为不知道应该做哪些单元测试,同时也是因为以前没做过 😂

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.

1 participant