-
Notifications
You must be signed in to change notification settings - Fork 67
Add ParallelStreamingDataset
#576
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
Merged
Merged
Changes from 31 commits
Commits
Show all changes
35 commits
Select commit
Hold shift + click to select a range
0c03590
Add ParallelStreamingDataset
philgzl aec7632
Add tests
philgzl a8cb189
Make __len__ an abstract method of base dataset wrapper
philgzl f81c25b
Update tests
philgzl 77e1847
Add num_cycles attribute
philgzl aeb6e3f
Finish cooking
philgzl adc97e6
Try to clean diff and add docs
philgzl 6e2c4fe
Fix mypy and 3.8 type hint errors
philgzl a5bb83a
Fix mypy errors for real this time
philgzl 376a28a
Merge branch 'main' into parallel-dataset
bhimrazy a7d941a
Merge branch 'main' into parallel-dataset
bhimrazy b9a21e4
Remove _BaseDatasetWrapperIterator
philgzl 2a19ff5
Update tests
philgzl bb34245
Fix RuntimeError when some workers are assigned 0 samples
philgzl 5131107
Fix mypy errors
philgzl b299b1b
Increase CI timeout from 35 to 45 minutes
deependujha 216ee0e
Update test_parallel.py to skip tests on macOS in addition to Windows
deependujha 65d0776
Add transform
philgzl 1f6563f
Update README.md
philgzl 6462652
Fix can't pickle local object error
philgzl 63d0bbe
Update README.md
philgzl 47aa5c5
Skip more tests on win32 and darwin
philgzl 2b92d46
Update docstrings
philgzl 673b716
Add comment in get_len
philgzl f49bd19
Replace tmpdir and tmdir_factory with tmp_path and tmp_path_factory
philgzl 9e06851
Merge branch 'main' into parallel-dataset
deependujha 696e471
Apply suggestions
philgzl f880bbd
Update tests
philgzl 1f6bebe
Update README.md
philgzl 3e81d4d
Merge branch 'main' into parallel-dataset
philgzl 59a52ad
Fix list type hint
philgzl ec4d706
Change samples_yieled to samples_yielded
philgzl f7c63a9
Skip even more tests to fit in macos CI time limit
philgzl b1a821a
Merge branch 'main' into parallel-dataset
philgzl 67c0269
Merge branch 'main' into parallel-dataset
bhimrazy File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.