Pytorch timeout=30

[farizrahman4u]

🚀 🚀 Pull Request

Checklist:

• My code follows the style guidelines of this project and the Contributing document
• I have commented my code, particularly in hard-to-understand areas
• I have kept the coverage-rate up
• I have performed a self-review of my own code and resolved any problems
• I have checked to ensure there aren't any other open Pull Requests for the same change
• I have described and made corresponding changes to the relevant documentation
• New and existing unit tests pass locally with my changes

Changes

Set pytorch dataloader timeout to 30 sec.

[AbhinavTuli]

can we make this an argument that the user passes and default to this value?

[farizrahman4u]

I think ideally there should be no timeout and the workers should be able to report exceptions to the main thread.. so better not change API right now.

[codecov]

Codecov Report

Merging #1520 (844eb04) into main (6ef9381) will decrease coverage by 0.03%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main    #1520      +/-   ##
==========================================
- Coverage   92.35%   92.31%   -0.04%     
==========================================
  Files         189      189              
  Lines       16453    16455       +2     
==========================================
- Hits        15195    15191       -4     
- Misses       1258     1264       +6     

Flag	Coverage Δ
unittests	92.31% <100.00%> (-0.04%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
hub/constants.py	100.00% <100.00%> (ø)
hub/integrations/pytorch/dataset.py	91.37% <100.00%> (-1.55%)	⬇️
hub/integrations/tests/test_pytorch_dataloader.py	95.69% <0.00%> (-2.16%)	⬇️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6ef9381...844eb04. Read the comment docs.