search

jaywalnut310 / vits

VITS: Conditional Variational Autoencoder with Adversarial Learning for End-to-End Text-to-Speech
https://jaywalnut310.github.io/vits-demo/index.html
MIT License
6.48k stars 1.21k forks source link

DistributedBucketSampler : create bucket sampler issue. #194

Open YoungJae98 opened 7 months ago

YoungJae98 commented 7 months ago

In DistributedBucketSampler's create_bucket function, there is a loop that detects and pops empty bucket.

The range is set from len(bucket)-1 to 0. In this range, first index is not referenced. Then error can occur if the first part is empty.

I wonder if I got it wrong, if not with any other intentions.

YoungJae98 commented 7 months ago

영어로 적다보니 의도가 다르게 전달됐을까 하여 한글로도 남깁니다..! distributed bucket sampler에서 bucket pop하는 부분에 현재처럼 0까지만 돌게되면 index 1까지만 순회하게 되는데, 그렇게되면 index 0일 경우에 비어있다면 비우지 못해서 다음 구간에서 오류가 발생할 수 있는것이 아닌가 하는 생각이 들어서 혹시 다른 의도가 있으셨던건지, 아니라면 제가 잘못 이해한건지 여쭤보고싶어 이슈로 남겼습니다!