Skip to content

Conversation

@letterx
Copy link

@letterx letterx commented Apr 15, 2020

Summary: If torch compiled without MKL, this test fails with torch.fft requiring MKL support

Test Plan: CI

Differential Revision: D21051362

@facebook-github-bot
Copy link
Contributor

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

@dr-ci
Copy link

dr-ci bot commented Apr 15, 2020

💊 Build failures summary and remediations

As of commit 3a0a3c8 (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 on the GitHub issue tracker.

See how this bot performed.

This comment has been revised 19 times.

@letterx letterx requested a review from malfet April 15, 2020 22:57
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
@unittest.skipIf(not TEST_MKL, "PyTorch is built without MKL support")
@skipCPUIfNoMkl()

@facebook-github-bot
Copy link
Contributor

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

@facebook-github-bot
Copy link
Contributor

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

)

Summary:
Pull Request resolved: pytorch#36680

If torch compiled without MKL, this test fails with torch.fft requiring MKL support

Test Plan: CI

Reviewed By: malfet

Differential Revision: D21051362

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

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

@facebook-github-bot
Copy link
Contributor

This pull request has been merged in b889e0d.

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.

5 participants