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

Update Developing Markdown #1617

Merged
merged 2 commits into from
Jun 9, 2023
Merged

Update Developing Markdown #1617

merged 2 commits into from
Jun 9, 2023

Conversation

rahul-tuli
Copy link
Member

Add note for running torch tests

@rahul-tuli rahul-tuli requested review from bfineran and dsikka June 9, 2023 15:27
@rahul-tuli rahul-tuli self-assigned this Jun 9, 2023
dsikka
dsikka previously approved these changes Jun 9, 2023
@rahul-tuli rahul-tuli added the documentation Improvements or additions to documentation label Jun 9, 2023
DEVELOPING.md Outdated
@@ -40,6 +40,9 @@ Those can be installed by adding the framework name to the install extras. Frame
python3 -m pip install -e "./[dev,torch]"
```

Note: Running all pytorch tests using `make test TARGETS=torch`, also requires `torchvision`
and `onnxruntime` install all these dependencies using `python3 -m pip install "./[dev, torch, torchvision, onnxruntime]"`
Copy link
Contributor

Choose a reason for hiding this comment

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

Missing -e?

Copy link
Member Author

Choose a reason for hiding this comment

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

Oooh Good catch Ben!

Copy link
Member Author

Choose a reason for hiding this comment

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

Updated!

@bfineran bfineran merged commit 24471da into main Jun 9, 2023
@bfineran bfineran deleted the update-developing-readme branch June 9, 2023 22:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants