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

UT RE - enable for timer util #506

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

Conversation

galrotem
Copy link
Contributor

Summary:

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: D48475432

@facebook-github-bot
Copy link
Contributor

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

@facebook-github-bot
Copy link
Contributor

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

galrotem added a commit to galrotem/tnt that referenced this pull request Aug 18, 2023
Summary:
Pull Request resolved: pytorch#506

# 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: D48475432

fbshipit-source-id: 110a21e6dc08eaadb7ddd5055385193d9ea9ef1b
@facebook-github-bot
Copy link
Contributor

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

galrotem added a commit to galrotem/tnt that referenced this pull request Aug 19, 2023
Summary:
Pull Request resolved: pytorch#506

# 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: D48475432

fbshipit-source-id: 1c4ec8fab84f80fc96bb03e0e0ea24ca7906dabc
@facebook-github-bot
Copy link
Contributor

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

galrotem added a commit to galrotem/tnt that referenced this pull request Aug 19, 2023
Summary:
Pull Request resolved: pytorch#506

# 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: D48475432

fbshipit-source-id: 607d36d9596b964e59143374f8b92ae4ca9f9b8b
@facebook-github-bot
Copy link
Contributor

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

galrotem added a commit to galrotem/tnt that referenced this pull request Aug 19, 2023
Summary:
Pull Request resolved: pytorch#506

# 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: D48475432

fbshipit-source-id: 7d4e4f0b33b85bc215534e142628c111ef0edb7d
Summary:
Pull Request resolved: pytorch#506

# 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: D48475432

fbshipit-source-id: 072f66e215fc261871f5bcbf90f93120f50fa5e5
@facebook-github-bot
Copy link
Contributor

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

@codecov
Copy link

codecov bot commented Aug 21, 2023

Codecov Report

Merging #506 (c79d026) into master (2bcc5ca) will decrease coverage by 0.04%.
The diff coverage is 86.20%.

@@            Coverage Diff             @@
##           master     #506      +/-   ##
==========================================
- Coverage   86.95%   86.92%   -0.04%     
==========================================
  Files         106      106              
  Lines        8407     8395      -12     
==========================================
- Hits         7310     7297      -13     
- Misses       1097     1098       +1     
Files Changed Coverage Δ
torchtnt/utils/env.py 84.90% <40.00%> (-6.01%) ⬇️
tests/utils/test_timer.py 92.98% <94.11%> (+0.98%) ⬆️
tests/utils/test_env.py 96.05% <100.00%> (+0.05%) ⬆️

📣 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