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

DataLoader2 with multiprocess raise exception: Can not request next item while we are still waiting response for previous request #1228

Open
npuichigo opened this issue Mar 12, 2024 · 4 comments

Comments

@npuichigo
Copy link

npuichigo commented Mar 12, 2024

🐛 Describe the bug

dp = ...
dp = dp.sharding_filter()
rs = MultiProcessingReadingService(num_workers=4)
dataloader = DataLoader2(dp, reading_service=rs)

for _ in dataloader:
    pass

dataloader.shutdown()
Exception: Can not request next item while we are still waiting response for previous request
This exception is thrown by __iter__ of _IterateQueueDataPipes(datapipes=[QueueWrapper, QueueWrapper, QueueWrapper, QueueWrapper])

Versions

[pip3] numpy==1.26.4
[pip3] onnx==1.15.0
[pip3] onnxconverter-common==1.13.0
[pip3] onnxruntime==1.15.1
[pip3] skl2onnx==1.16.0
[pip3] torch==2.2.1
[pip3] torchaudio==2.2.1
[pip3] torchdata==0.7.1
[pip3] torchvision==0.17.1
[pip3] triton==2.2.0
[conda] numpy                     1.26.4                   pypi_0    pypi
[conda] torch                     2.2.1                    pypi_0    pypi
[conda] torchaudio                2.2.1                    pypi_0    pypi
[conda] torchvision               0.17.1                   pypi_0    pypi
[conda] triton
@jdenhof
Copy link

jdenhof commented Mar 26, 2024

I have noticed this happens whenever the number of workers you specify for the MultiProcessingReadingService is greater than then the number elements that can be yielded from the dp before sharding.

@qmpzzpmq
Copy link

qmpzzpmq commented Nov 7, 2024

@jdenhof I chcek it, even with he number of workers specify for the MultiProcessingReadingService is smaller than the number elements that can be yielded from the dp before sharding. Still have this issue.

@ds2268
Copy link

ds2268 commented Nov 9, 2024

I also noticed this issue at the end/start of the first epoch. Any fix?

@qmpzzpmq
Copy link

@ds2268 I fixed it by #1311

SarthakYadav added a commit to SarthakYadav/data that referenced this issue Dec 18, 2024
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

4 participants