-
Notifications
You must be signed in to change notification settings - Fork 1.2k
Standardized behavior of Batch.cat and misc code refactor #137
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 all commits
Commits
Show all changes
25 commits
Select commit
Hold shift + click to select a range
1f84806
code refactor; remove unused kwargs; add reward_normalization for dqn
youkaichao 80a4f98
bugfix for __setitem__ with torch.Tensor; add Batch.condense
youkaichao 3d7cc24
minor fix
Trinkle23897 fc84433
support cat with empty Batch
youkaichao 9fa118c
remove the dependency of is_empty on len; specify the semantic of emp…
youkaichao ceca419
support stack with empty Batch
youkaichao 4557baa
remove condense
youkaichao 3de0218
refactor code to reflect the shared / partial / reserved categories o…
youkaichao f840c73
add is_empty(recursive=False)
youkaichao ce08bac
doc fix
youkaichao 35b1533
docfix and bugfix for _is_batch_set
youkaichao 8c2847f
add doc for key reservation
youkaichao e1e36e0
bugfix for algebra operators
youkaichao 6d2cda6
fix cat with lens hint
youkaichao ebf19ea
code refactor
youkaichao 2541bec
bugfix for storing None
youkaichao 28b33e8
use ValueError instead of exception
youkaichao 3046067
hide lens away from users
youkaichao fba94a6
add comment for __cat
youkaichao 6287326
move the computation of the initial value of lens in cat_ itself.
youkaichao f64faf5
change the place of doc string
youkaichao b795493
doc fix for Batch doc string
youkaichao 02f9d1a
change recursive to recurse
youkaichao 2a063f0
doc string fix
youkaichao 7912eae
minor fix for batch doc
Trinkle23897 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
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.