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

[Docs] Readme update #19

Merged
merged 16 commits into from
Oct 2, 2023
Merged

[Docs] Readme update #19

merged 16 commits into from
Oct 2, 2023

Conversation

matteobettini
Copy link
Collaborator

@matteobettini matteobettini commented Sep 27, 2023

Started filling README contents and organizing examples

Signed-off-by: Matteo Bettini <[email protected]>
Signed-off-by: Matteo Bettini <[email protected]>
Signed-off-by: Matteo Bettini <[email protected]>
Signed-off-by: Matteo Bettini <[email protected]>
# Conflicts:
#	premade_scripts/vmas/config.yaml
Signed-off-by: Matteo Bettini <[email protected]>
Signed-off-by: Matteo Bettini <[email protected]>
Signed-off-by: Matteo Bettini <[email protected]>
@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 Sep 27, 2023
Comment on lines +53 to +57
pip install git+https://github.com/pytorch-labs/tensordict
git clone https://github.com/pytorch/rl.git
cd rl
python setup.py develop
cd ..
Copy link
Collaborator Author

@matteobettini matteobettini Sep 27, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@vmoens
This is currently the only way i am able to install torchrl nightly.

This is because of many issues:

  • the MacOS compatibility prevents from using the nightly dependency
  • the torchrl._torchrl makes it so that pip install git+https://github.com/pytorch/rl.git and git clone + pip install -e . do not work

Telling the users to clone rl seems a lot to me, is there any workaround?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

sorry i missed your tag
oh yes I know about that, there's an easy fix, let me take care of it

Signed-off-by: Matteo Bettini <[email protected]>
Signed-off-by: Matteo Bettini <[email protected]>
Signed-off-by: Matteo Bettini <[email protected]>
Signed-off-by: Matteo Bettini <[email protected]>
Signed-off-by: Matteo Bettini <[email protected]>
Signed-off-by: Matteo Bettini <[email protected]>
Signed-off-by: Matteo Bettini <[email protected]>
@matteobettini matteobettini marked this pull request as ready for review October 2, 2023 08:27
@matteobettini matteobettini changed the title Readme [Readme] Update Oct 2, 2023
@matteobettini matteobettini changed the title [Readme] Update [Docs] Readme update Oct 2, 2023
@matteobettini matteobettini merged commit 7afff7d into main Oct 2, 2023
9 of 12 checks passed
@matteobettini matteobettini deleted the readme branch October 2, 2023 08:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
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.

3 participants