-
-
Notifications
You must be signed in to change notification settings - Fork 77
Support PyArrow #202
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
Support PyArrow #202
Changes from 22 commits
Commits
Show all changes
28 commits
Select commit
Hold shift + click to select a range
471f32e
add arrow storage
younik 9748999
use two different methods for info/spaces
younik 126873a
fix info shape
younik fb83f97
Merge remote-tracking branch 'farama/main' into pyarrow
younik b5959b1
add deps
younik 7808516
fix pre-commit
younik 27444a9
fix None seed
younik 054c0cc
fix docs
younik ccad227
fix minari show
younik ffe5661
fix docs
younik 5c73ca9
remove max_buffer steps
younik 76d6bda
add EpisodeBuffer utility
younik 79be5c4
fix tests
younik e35e788
fix docs
younik f8c1ec7
add docstring
younik eed690d
fix h5py storage
younik 4db153f
reformat
younik b7d97b5
test both storages
younik 8f800a9
improve tests
younik cf4fee3
reformat
younik 084c3d2
use arrow instead of parquet
younik 159bff8
change default to hdf5
younik 2e99814
refactor data_collector
younik 52bcf75
add typing to EpisodeBuffer
younik b28a81b
address comments
younik 43d5187
add data_format to tests
younik 9589b51
fix dataset combine test
younik 776cd93
improve arrow efficiency
younik 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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,4 +1,6 @@ | ||
| from minari.data_collector.callbacks import StepData | ||
| from minari.data_collector.data_collector import DataCollector | ||
| from minari.data_collector.episode_buffer import EpisodeBuffer | ||
|
|
||
|
|
||
| __all__ = ["DataCollector"] | ||
| __all__ = ["DataCollector", "StepData", "EpisodeBuffer"] | ||
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.
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.