-
Notifications
You must be signed in to change notification settings - Fork 328
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
[Feature] step_and_maybe_reset in env #1611
Merged
Merged
Changes from 1 commit
Commits
Show all changes
122 commits
Select commit
Hold shift + click to select a range
1f539dd
init
vmoens d4c16e1
amend
vmoens d2321aa
amend
vmoens 565115a
amend
vmoens 3c46136
amend
vmoens 78cfa41
amend
vmoens a6bd8eb
amend
vmoens 04d4ae7
amend
vmoens 2d0b4c6
init
vmoens 528609a
amend
vmoens 51cd5af
fix
vmoens 3e31963
Merge remote-tracking branch 'origin/main' into step_maybe_reset
vmoens b11e73a
amend
vmoens 1540407
remove `pip3 install -e .`
vmoens f1b0ea4
tensordict_
vmoens 16b3538
amend rollout logic
vmoens 7ad0864
amend
vmoens bcac398
amend
vmoens 428f8ee
inference
vmoens 6fbd0bd
cpu -> cuda
vmoens 02db623
checks
vmoens 08a8f47
using pipe instead of event
vmoens 45e64f7
amend
vmoens 7dd4821
amend
vmoens e1a2206
rm cuda event
vmoens dc2caab
amend
vmoens 01ffbf9
amend
vmoens ac76ec3
amend
vmoens ceab010
amend
vmoens f0327c9
amend
vmoens 518b3d1
amend
vmoens 47dd93b
amend
vmoens 354fb6f
amend
vmoens 53d5f9a
amend
vmoens 78c00e8
amend
vmoens f63480e
amend
vmoens 9a3631f
amend
vmoens 44336ed
Merge remote-tracking branch 'origin/main' into fix_ci
vmoens 512f9f7
specs fix
vmoens 2ceb438
amend
vmoens c666d5a
amend
vmoens 6ecebda
amend
vmoens ce6e9bd
amend
vmoens 5c613c3
amend
vmoens 9f97e58
amend
vmoens 9cbcbb0
amend
vmoens ae2748d
amend
vmoens 72c4163
amend
vmoens 6f4c374
amend
vmoens bf36bec
amend
vmoens 4095766
amend
vmoens 9f2a9ad
amend
vmoens 5b34961
amend
vmoens deba78d
amend
vmoens b638461
amend
vmoens f8b2f60
amend
vmoens dfc868a
amend
vmoens 1221c48
Merge branch 'fix_ci' into step_maybe_reset
vmoens 22d3a27
amend
vmoens 38e0e90
amend
vmoens 5828a84
Merge branch 'fix_ci' into step_maybe_reset
vmoens 004d381
amend
vmoens 6ef720d
amend
vmoens d2167ea
Merge branch 'fix_ci' into step_maybe_reset
vmoens f66c6c9
amend
vmoens 4fd2768
amend
vmoens ded1883
amend
vmoens cb1a83f
amend
vmoens 6af6a45
amend
vmoens eaf5ebd
amend
vmoens f426322
Merge branch 'fix_ci' into step_maybe_reset
vmoens 83672c6
amend
vmoens f0a6134
amend
vmoens c589a4a
amend
vmoens a85b321
amend
vmoens ed5e96f
amend
vmoens e705719
amend
vmoens 4c3fa33
fix
vmoens e354bc8
fix
vmoens faa3b41
amend
vmoens 80120b4
fix
vmoens 09205e5
empty
vmoens 3c41dab
amend
vmoens 4db8110
amend
vmoens 38cac91
empty
vmoens a336f0e
Merge remote-tracking branch 'origin/main' into step_maybe_reset
vmoens a9f4678
amend
vmoens 9ee6348
amend
vmoens 62c848b
amend
vmoens 684d527
amend
vmoens 89ddfd2
amend
vmoens a0376e4
amend
vmoens 44dfe86
amend
vmoens d332597
amend
vmoens 85cf664
amend
vmoens b55fae1
Merge remote-tracking branch 'origin/main' into step_maybe_reset
vmoens 5dcbf15
amend
vmoens 0236818
amend
vmoens 36f151d
amend
vmoens afecc65
amend
vmoens f4a0beb
amend
vmoens 5a331e5
amend
vmoens 1ee4ba7
amend
vmoens 9d14427
amend
vmoens 8c15741
amend
vmoens 1fe2cd3
amend
vmoens c378acc
amend
vmoens 551c9eb
amend
vmoens e177c77
amend
vmoens 28b0059
amend
vmoens 5a21f2a
amend
vmoens b7b2081
lint
vmoens de1ecf2
amend
vmoens 6fec99e
amend
vmoens 6bfe517
amend
vmoens 8f78ac0
cache keys
vmoens d2b734b
fix empty cache
vmoens a0c12d5
Merge remote-tracking branch 'origin/main' into step_maybe_reset
vmoens 83eb52f
amend
vmoens 5943c51
Merge remote-tracking branch 'origin/main' into step_maybe_reset
vmoens 3e86847
addressing comments
vmoens ca1dd78
amend
vmoens 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
lint
- Loading branch information
commit b7b2081af094d948efb8a52855e5b44ce627e552
There are no files selected for viewing
This file contains 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 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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This crashes when
tensordict is None
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah sorry about that, I can't test petting zoo locally so I'm always moving in the dark...