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

SSNR is 10.4 #23

Open
DingChun-code opened this issue Mar 2, 2023 · 5 comments
Open

SSNR is 10.4 #23

DingChun-code opened this issue Mar 2, 2023 · 5 comments

Comments

@DingChun-code
Copy link

I used your open-source code in github, ran evaluation.py, and also downloaded VCTK+DEMAND data. The model in it uses the model checkpoint you left on github in best_ckpt folder. Because the source data is 48000hz, I added the resampling part in the code. After running evaluation.py, the SSNR of the result is only 10.4, and other indicators are also slightly lower than the paper. If you have any opinions, I would appreciate it very much.

@utyanndesu
Copy link

We encountered the same problem, with the SSNR indicator only 10.4 and each indicator slightly lower

@BancoLin
Copy link

BancoLin commented May 19, 2023

report my results:

  • use ffmpeg to downsample the source data from 48khz to 16khz.
  • use pretrained model in the best_ckpt/ folder
  • use CPU to do evaluation

In the following metrics the first numbers are from the TABLE I of the paper (https://arxiv.org/pdf/2209.11112.pdf), and the second ones are my results. All the results are similar except the SSNR.

PESQ: 3.41 --> 3.41
CSIG: 4.63 --> 4.67
CBAK: 3.94 --> 3.91
COVL: 4.12 --> 4.14
SSNR: 11.10 --> 10.68
STOI: 0.96 --> 0.96

despite the inconsistent numbers between code and paper, i'm quite satisfied the performance with our own data.

@SherifAbdulatif
Copy link
Collaborator

We got some complains about SSNR and it is currently being checked.

@moshengmao
Copy link

  • use ffmpeg to downsample the source data from 48khz to 16khz

I want to ask if I use the VCTK-DEMAND dataset to train, whether I need downsample the trainset to 16khz first or input the original data.

@SherifAbdulatif
Copy link
Collaborator

SherifAbdulatif commented Nov 17, 2023 via email

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

5 participants