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

make ddp logging api to be private #57999

Closed
wants to merge 1 commit into from

Conversation

zhaojuanmao
Copy link
Contributor

@zhaojuanmao zhaojuanmao commented May 10, 2021

Stack from ghstack:

make ddp logging api to be private

Differential Revision: D28338485

make ddp logging api to be private

Differential Revision: [D28338485](https://our.internmc.facebook.com/intern/diff/D28338485/)

[ghstack-poisoned]
@facebook-github-bot facebook-github-bot added the oncall: distributed Add this issue/PR to distributed oncall triage queue label May 10, 2021
@facebook-github-bot
Copy link
Contributor

facebook-github-bot commented May 10, 2021

💊 CI failures summary and remediations

As of commit da28845 (more details on the Dr. CI page):


💚 💚 Looks good so far! There are no failures yet. 💚 💚


This comment was automatically generated by Dr. CI (expand for details).Follow this link to opt-out of these comments for your Pull Requests.

Please report bugs/suggestions to the (internal) Dr. CI Users group.

Click here to manually regenerate this comment.

zhaojuanmao added a commit that referenced this pull request May 10, 2021
make ddp logging api to be private

Differential Revision: [D28338485](https://our.internmc.facebook.com/intern/diff/D28338485/)

ghstack-source-id: 128607185
Pull Request resolved: #57999
Copy link
Member

@rohan-varma rohan-varma left a comment

Choose a reason for hiding this comment

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

lgtm, thanks!

@codecov
Copy link

codecov bot commented May 11, 2021

Codecov Report

Merging #57999 (da28845) into gh/zhaojuanmao/75/base (807bea1) will increase coverage by 0.01%.
The diff coverage is 13.33%.

@@                    Coverage Diff                     @@
##           gh/zhaojuanmao/75/base   #57999      +/-   ##
==========================================================
+ Coverage                   76.83%   76.85%   +0.01%     
==========================================================
  Files                        1986     1986              
  Lines                      197417   197417              
==========================================================
+ Hits                       151681   151716      +35     
+ Misses                      45736    45701      -35     

@facebook-github-bot
Copy link
Contributor

This pull request has been merged in ac44569.

@facebook-github-bot
Copy link
Contributor

This pull request has been reverted by ad4cd6e.

@facebook-github-bot facebook-github-bot deleted the gh/zhaojuanmao/75/head branch May 15, 2021 14:23
krshrimali pushed a commit to krshrimali/pytorch that referenced this pull request May 19, 2021
Summary:
Pull Request resolved: pytorch#57999

make ddp logging api to be private
ghstack-source-id: 128607185

Test Plan: unit test

Reviewed By: rohan-varma

Differential Revision: D28338485

fbshipit-source-id: bd2ae7c78904e93eed88be91876f5a832b5b7886
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 Reverted
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants