Skip to content
This repository was archived by the owner on Aug 7, 2024. It is now read-only.

Conversation

@drisspg
Copy link
Contributor

@drisspg drisspg commented Dec 18, 2023

Summary

Moved depdencies that are only used for tests or for benchmarking to optional depdency

Also fix format

Usage would be
pip install - e .[test]

@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 Dec 18, 2023
@facebook-github-bot
Copy link
Contributor

@drisspg has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

@facebook-github-bot
Copy link
Contributor

@drisspg has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

@drisspg drisspg requested a review from vkuzo December 18, 2023 01:51
Copy link
Contributor

@vkuzo vkuzo left a comment

Choose a reason for hiding this comment

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

maybe update the readme too?

@drisspg drisspg force-pushed the updated_torch_dependency branch from d699e59 to c4a2599 Compare December 20, 2023 02:04
@facebook-github-bot
Copy link
Contributor

@drisspg has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

@facebook-github-bot
Copy link
Contributor

@drisspg merged this pull request in b41006b.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. Merged

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants