Skip to content
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

[BugFix] Fix Examples #687

Merged
merged 23 commits into from
Nov 21, 2022
Merged

[BugFix] Fix Examples #687

merged 23 commits into from
Nov 21, 2022

Conversation

vmoens
Copy link
Contributor

@vmoens vmoens commented Nov 18, 2022

Description

Fixes examples and introduces an example test pipeline.

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Nov 18, 2022
@vmoens vmoens added bug Something isn't working CI Has to do with CI setup (e.g. wheels & builds, tests...) labels Nov 18, 2022
@codecov
Copy link

codecov bot commented Nov 19, 2022

Codecov Report

Merging #687 (661f151) into main (f5d98af) will increase coverage by 0.71%.
The diff coverage is 87.50%.

@@            Coverage Diff             @@
##             main     #687      +/-   ##
==========================================
+ Coverage   87.92%   88.64%   +0.71%     
==========================================
  Files         120      121       +1     
  Lines       20512    20670     +158     
==========================================
+ Hits        18036    18323     +287     
+ Misses       2476     2347     -129     
Flag Coverage Δ
habitat-gpu 24.08% <21.87%> (+0.02%) ⬆️
linux-cpu 84.71% <85.10%> (-0.02%) ⬇️
linux-gpu 85.60% <85.10%> (-0.03%) ⬇️
linux-jumanji 29.06% <21.87%> (+0.02%) ⬆️
linux-outdeps-gpu 72.06% <84.44%> (-0.02%) ⬇️
linux-stable-cpu 84.57% <85.10%> (-0.02%) ⬇️
linux-stable-gpu 85.29% <85.10%> (-0.02%) ⬇️
linux_examples-gpu 42.95% <61.53%> (?)
macos-cpu 84.39% <85.10%> (-0.02%) ⬇️
olddeps-gpu 74.35% <84.44%> (-0.02%) ⬇️

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

Impacted Files Coverage Δ
torchrl/trainers/helpers/replay_buffer.py 94.44% <ø> (+22.22%) ⬆️
examples/dreamer/dreamer_utils.py 78.28% <50.00%> (+27.42%) ⬆️
torchrl/trainers/helpers/envs.py 74.87% <50.00%> (+15.70%) ⬆️
torchrl/envs/transforms/transforms.py 86.69% <83.33%> (+0.65%) ⬆️
torchrl/data/replay_buffers/storages.py 83.84% <90.47%> (+0.19%) ⬆️
examples/dreamer/dreamer.py 87.66% <100.00%> (ø)
test/test_rb.py 96.42% <100.00%> (-0.05%) ⬇️
test/test_trainer.py 97.92% <100.00%> (-0.02%) ⬇️
torchrl/data/replay_buffers/rb_prototype.py 87.40% <100.00%> (-0.37%) ⬇️
torchrl/data/replay_buffers/replay_buffers.py 78.88% <100.00%> (+4.94%) ⬆️
... and 12 more

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@vmoens vmoens merged commit a3bbba0 into main Nov 21, 2022
vmoens added a commit that referenced this pull request Nov 23, 2022
* amend

* [BugFix] ConvNet forward method with tensors of more than 4 dimensions (#686)

* cnn forward fix

* more general code

* cnn testing

* precommit run check

* convnet tests

* [Feature] add `standard_normal` for RewardScaling (#682)

* Add standard_normal

* give attribute access

* Update standard_normal

* Update tests

* Fix tests

* Address in-place scaling of reward

* Improvise tests

* [Feature] Jumanji envs (#674)

* amend

* [Feature] Default collate_fn (#688)

* init

* amend

* amend

* [BugFix] Fix Examples (#687)

* amend

* [Refactoring] Replace direct gym version checks with decorated functions (#691)

* [Refactoring] Replace gym version checking with decorated functions (#)

Initial commit. Only tests.

* Refactoring in gym.py

* More refactoring in gym.py

* Completed refactoring

* amend

* amend

* Version 0.0.3 (#696)

* [Docs] Host TensorDict docs inside TorchRL docs (#693)

* Pull tensordict docs into TorchRL docs

* Add banner for tensordict docs

* [BugFix] Fix docs build (#698)

* [BugFix] Proper error messages for orphan transform creation (#697)

* amend

* [Feature] Append, init and insert transforms in ReplayBuffer (#695)

* lint

Co-authored-by: albertbou92 <albertbou92@users.noreply.github.com>
Co-authored-by: Aditya Gandhamal <61016383+adityagandhamal@users.noreply.github.com>
Co-authored-by: yingchenlin <yc.jon.lin@gmail.com>
Co-authored-by: Sergey Ordinskiy <113687736+ordinskiy@users.noreply.github.com>
Co-authored-by: Tom Begley <tomcbegley@gmail.com>
Co-authored-by: Alan Schelten <alan@schelten.net>
@vmoens vmoens deleted the fix_examples branch November 24, 2022 08:44
vmoens added a commit that referenced this pull request Jan 5, 2023
* init

* tests1

* run examples in tests

* [Feature] MPPI Planner (#694)

* amend

* [BugFix] ConvNet forward method with tensors of more than 4 dimensions (#686)

* cnn forward fix

* more general code

* cnn testing

* precommit run check

* convnet tests

* [Feature] add `standard_normal` for RewardScaling (#682)

* Add standard_normal

* give attribute access

* Update standard_normal

* Update tests

* Fix tests

* Address in-place scaling of reward

* Improvise tests

* [Feature] Jumanji envs (#674)

* amend

* [Feature] Default collate_fn (#688)

* init

* amend

* amend

* [BugFix] Fix Examples (#687)

* amend

* [Refactoring] Replace direct gym version checks with decorated functions (#691)

* [Refactoring] Replace gym version checking with decorated functions (#)

Initial commit. Only tests.

* Refactoring in gym.py

* More refactoring in gym.py

* Completed refactoring

* amend

* amend

* Version 0.0.3 (#696)

* [Docs] Host TensorDict docs inside TorchRL docs (#693)

* Pull tensordict docs into TorchRL docs

* Add banner for tensordict docs

* [BugFix] Fix docs build (#698)

* [BugFix] Proper error messages for orphan transform creation (#697)

* amend

* [Feature] Append, init and insert transforms in ReplayBuffer (#695)

* lint

Co-authored-by: albertbou92 <albertbou92@users.noreply.github.com>
Co-authored-by: Aditya Gandhamal <61016383+adityagandhamal@users.noreply.github.com>
Co-authored-by: yingchenlin <yc.jon.lin@gmail.com>
Co-authored-by: Sergey Ordinskiy <113687736+ordinskiy@users.noreply.github.com>
Co-authored-by: Tom Begley <tomcbegley@gmail.com>
Co-authored-by: Alan Schelten <alan@schelten.net>

* lint

Co-authored-by: albertbou92 <albertbou92@users.noreply.github.com>
Co-authored-by: Aditya Gandhamal <61016383+adityagandhamal@users.noreply.github.com>
Co-authored-by: yingchenlin <yc.jon.lin@gmail.com>
Co-authored-by: Sergey Ordinskiy <113687736+ordinskiy@users.noreply.github.com>
Co-authored-by: Tom Begley <tomcbegley@gmail.com>
Co-authored-by: Alan Schelten <alan@schelten.net>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working CI Has to do with CI setup (e.g. wheels & builds, tests...) CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants