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

[torch vitals] Initial implementation #51047

Closed
wants to merge 1 commit into from

Conversation

bwasti
Copy link
Contributor

@bwasti bwasti commented Jan 25, 2021

Summary:
If the environment variable TORCH_VITAL is set to a non-zero length string, the vitals a dumped at program end.

The API is very similar to google's logging

Test Plan: buck test //caffe2/aten:vitals

Reviewed By: bitfort

Differential Revision: D25791248

Summary:
If the environment variable `TORCH_VITAL` is set to a non-zero length string, the vitals a dumped at program end.

The API is very similar to google's logging

Test Plan: buck test //caffe2/aten:vitals

Reviewed By: bitfort

Differential Revision: D25791248

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

facebook-github-bot commented Jan 25, 2021

💊 CI failures summary and remediations

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


  • 2/2 failures possibly* introduced in this PR
    • 2/2 non-CircleCI failure(s)

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.

@facebook-github-bot
Copy link
Contributor

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

@codecov
Copy link

codecov bot commented Jan 26, 2021

Codecov Report

Merging #51047 (945081e) into master (ac0a3cc) will decrease coverage by 0.00%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master   #51047      +/-   ##
==========================================
- Coverage   80.92%   80.91%   -0.01%     
==========================================
  Files        1926     1928       +2     
  Lines      210019   210033      +14     
==========================================
+ Hits       169950   169953       +3     
- Misses      40069    40080      +11     

@facebook-github-bot
Copy link
Contributor

This pull request has been merged in 069602e.

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.

None yet

2 participants