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

pyre - eliminate errors from distributed #529

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

galrotem
Copy link
Contributor

@galrotem galrotem commented Sep 1, 2023

Summary:
Get rid of pyre error annotations in distributed util

Internal

Context

[Describe motivations and existing situation that led to creating this diff. Don't be cheap with context, it is the basis for a good code review.]

This diff

[List all the changes that this diff introduces and explain the ones that are not trivial. Give directions for the reviewer if needed.]

What’s next

[If this diff is part of a stack or if it has direct continuation in a future diff, share these plans with your reviewer.]

Differential Revision: D48872019

Summary:
Get rid of pyre error annotations in distributed util

Internal


# Context
[Describe motivations and existing situation that led to creating this diff. Don't be cheap with context, it is the basis for a good code review.]


# This diff
[List all the changes that this diff introduces and explain the ones that are not trivial. Give directions for the reviewer if needed.]


# What’s next
[If this diff is part of a stack or if it has direct continuation in a future diff, share these plans with your reviewer.]

Differential Revision: D48872019
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D48872019

@codecov
Copy link

codecov bot commented Sep 1, 2023

Codecov Report

Merging #529 (5b3fb54) into master (1f703d5) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master     #529   +/-   ##
=======================================
  Coverage   86.31%   86.32%           
=======================================
  Files          61       61           
  Lines        3873     3875    +2     
=======================================
+ Hits         3343     3345    +2     
  Misses        530      530           
Files Changed Coverage Δ
torchtnt/utils/distributed.py 76.58% <100.00%> (+0.11%) ⬆️
torchtnt/utils/timer.py 94.59% <100.00%> (+0.02%) ⬆️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

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.

2 participants