-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
DistributedBucketSampler : create bucket sampler issue. #194
Comments
영어로 적다보니 의도가 다르게 전달됐을까 하여 한글로도 남깁니다..! distributed bucket sampler에서 bucket pop하는 부분에 현재처럼 0까지만 돌게되면 index 1까지만 순회하게 되는데, 그렇게되면 index 0일 경우에 비어있다면 비우지 못해서 다음 구간에서 오류가 발생할 수 있는것이 아닌가 하는 생각이 들어서 혹시 다른 의도가 있으셨던건지, 아니라면 제가 잘못 이해한건지 여쭤보고싶어 이슈로 남겼습니다! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
The text was updated successfully, but these errors were encountered: