Skip to content

Add note about TCP init in RPC tests to contributing doc.#50861

Closed
rohan-varma wants to merge 3 commits intopytorch:masterfrom
rohan-varma:contrib_doc
Closed

Add note about TCP init in RPC tests to contributing doc.#50861
rohan-varma wants to merge 3 commits intopytorch:masterfrom
rohan-varma:contrib_doc

Conversation

@rohan-varma
Copy link
Copy Markdown
Contributor

We added this option in #48248, but it would be good to document it somewhere as well, hence adding it to this contributing doc.

@facebook-github-bot facebook-github-bot added cla signed oncall: distributed Add this issue/PR to distributed oncall triage queue labels Jan 21, 2021
@rohan-varma rohan-varma requested review from H-Huang and wayi1 January 21, 2021 00:28
@codecov
Copy link
Copy Markdown

codecov Bot commented Jan 21, 2021

Codecov Report

Merging #50861 (cdfcc3b) into master (7fdc6a2) will increase coverage by 5.36%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master   #50861      +/-   ##
==========================================
+ Coverage   75.65%   81.01%   +5.36%     
==========================================
  Files        1916     1916              
  Lines      209287   209287              
==========================================
+ Hits       158343   169561   +11218     
+ Misses      50944    39726   -11218     

@wayi1
Copy link
Copy Markdown
Contributor

wayi1 commented Jan 21, 2021

Thanks for the documentation!

Copy link
Copy Markdown
Contributor

@facebook-github-bot facebook-github-bot left a comment

Choose a reason for hiding this comment

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

@rohan-varma has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

@facebook-github-bot
Copy link
Copy Markdown
Contributor

@rohan-varma merged this pull request in 7e10fbf.

laurentdupin pushed a commit to laurentdupin/pytorch that referenced this pull request Apr 24, 2026
)

Summary:
We added this option in pytorch#48248, but it would be good to document it somewhere as well, hence adding it to this contributing doc.

Pull Request resolved: pytorch#50861

Reviewed By: mrshenli

Differential Revision: D26014505

Pulled By: rohan-varma

fbshipit-source-id: c1321679f01dd52038131ff571362ad36884510a
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla signed Merged oncall: distributed Add this issue/PR to distributed oncall triage queue

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants