-
Notifications
You must be signed in to change notification settings - Fork 26.3k
utils: Only check for xnnpack if torch installed #74342
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
Conversation
Fixes a bug where collect_env.py was not able to be run without having torch installed Signed-off-by: Eli Uriegas <eliuriegas@fb.com>
CI Flow Status⚛️ CI FlowRuleset - Version:
|
🔗 Helpful links
💊 CI failures summary and remediationsAs of commit 25f2a49 (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).Please report bugs/suggestions to the (internal) Dr. CI Users group. |
|
@pytorchbot merge this |
|
Merge failed due to PR 74342 does not match merge rules |
|
|
@seemethere has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. |
Summary: Fixes a bug where collect_env.py was not able to be run without having torch installed Signed-off-by: Eli Uriegas <eliuriegas@fb.com> Pull Request resolved: #74342 Reviewed By: malfet, janeyx99 Differential Revision: D34943464 Pulled By: seemethere fbshipit-source-id: dbaa0004b88cb643a9c6426c9ea7c5be3d3c9ef5
This wasn't previously tested but this script is an essential tool that we use to help users present debug information on issues so we treat it with a similar level of care Depends on #74342 Signed-off-by: Eli Uriegas <eliuriegas@fb.com> Pull Request resolved: #74347 Approved by: https://github.com/janeyx99, https://github.com/malfet
Summary: This wasn't previously tested but this script is an essential tool that we use to help users present debug information on issues so we treat it with a similar level of care Depends on #74342 Signed-off-by: Eli Uriegas <eliuriegas@fb.com> Pull Request resolved: #74347 Approved by: https://github.com/janeyx99, https://github.com/malfet Test Plan: contbuild & OSS CI, see https://hud.pytorch.org/commit/pytorch/pytorch/abc868dd6446434dc519d3c5d9d210b2d406781c Reviewed By: malfet, osalpekar Differential Revision: D34968519 Pulled By: seemethere fbshipit-source-id: 59246cd1fbe4fe2ecdaeb5c1e4f988da57bee045
This wasn't previously tested but this script is an essential tool that we use to help users present debug information on issues so we treat it with a similar level of care Depends on #74342 Signed-off-by: Eli Uriegas <eliuriegas@fb.com> Pull Request resolved: #74347 Approved by: https://github.com/janeyx99, https://github.com/malfet

Fixes a bug where collect_env.py was not able to be run without having
torch installed
Signed-off-by: Eli Uriegas eliuriegas@fb.com