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

DOC Add instruction to install parametrized package #1164

Merged
merged 1 commit into from
Jan 8, 2021

Conversation

NicolasHug
Copy link
Member

@NicolasHug NicolasHug commented Jan 8, 2021

I couldn't find docs to install the package elsewhere (maybe I missed something?)

Also, it might be worth pointing to the testing READMEs directly from the root README, e.g. within the contributing guidelines. It took me a bit to find it ;). Happy to do that in this PR if needed.

@mthrok
Copy link
Collaborator

mthrok commented Jan 8, 2021

Yes, torchaudio needs CONTRIBUTING.md, which talks about testing too, but we do not have a bandwidth to work on. Are you interested in creating one? See also #926

@NicolasHug
Copy link
Member Author

Thanks for the review!

I'd be happy to write a CONTRIBUTING.md. I'll start that once I know a little bit more about the workflow - likely once my upcoming PR for #1059 is merged!

@NicolasHug NicolasHug merged commit d4fe76e into pytorch:master Jan 8, 2021
@NicolasHug NicolasHug deleted the parametrized_dep branch January 8, 2021 14:52
mthrok pushed a commit to mthrok/audio that referenced this pull request Feb 26, 2021
* Update tutorials to use TensorPipeRpcBackendOptions.

* Commit

Summary:

Test Plan:

Reviewers:

Subscribers:

Tasks:

Tags:

* Commit2

Summary:

Test Plan:

Reviewers:

Subscribers:

Tasks:

Tags:

Co-authored-by: pritam <pritam.damania@fb.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants