Nothing Special   »   [go: up one dir, main page]

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

Make unary tests runnable if SCIPY is not installed #59304

Conversation

malfet
Copy link
Contributor
@malfet malfet commented Jun 2, 2021

By adding if TEST_SCIPY else _NOTHING to special.i1 and special.i1e

Discovered while running tests on M1

By adding `if TEST_SCIPY else _NOTHING` to special.i1 and special.i1e
@malfet malfet requested review from mruberry and a team June 2, 2021 15:24
@facebook-github-bot
Copy link
Contributor
facebook-github-bot commented Jun 2, 2021

💊 CI failures summary and remediations

As of commit 53ca01d (more details on the Dr. CI page):


  • 1/1 failures possibly* introduced in this PR
    • 1/1 non-scanned 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.

Click here to manually regenerate this comment.

@facebook-github-bot
Copy link
Contributor

@malfet has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

Copy link
Collaborator
@mruberry mruberry left a comment

Choose a reason for hiding this comment

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

Nice catch!

@mruberry
Copy link
Collaborator
mruberry commented Jun 2, 2021

cc @kshitij12345

@kshitij12345
Copy link
Collaborator

Thanks!!

@facebook-github-bot
Copy link
Contributor

@malfet merged this pull request in f7097b0.

@malfet malfet deleted the malfet/make-test-unary-funcs-runnable-in-abscense-of-SCIPY branch June 2, 2021 20:35
deniskokarev pushed a commit to deniskokarev/pytorch that referenced this pull request Jun 9, 2021
Summary:
By adding `if TEST_SCIPY else _NOTHING` to special.i1 and special.i1e

Discovered while running tests on M1

Pull Request resolved: pytorch#59304

Reviewed By: jbschlosser

Differential Revision: D28835693

Pulled By: malfet

fbshipit-source-id: e4fde6584da29fa43bc6da75eebe560512754ed0
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.

4 participants