Skip to content

Conversation

@IvanYashchuk
Copy link
Collaborator

Added missing error checks for the CPU version of triangular_solve.
Fixes #43141.

@dr-ci
Copy link

dr-ci bot commented Sep 15, 2020

💊 CI failures summary and remediations

As of commit b9445f0 (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 or post in the (internal) Dr. CI Users group.

See how this bot performed.

This comment has been revised 6 times.

@codecov
Copy link

codecov bot commented Sep 16, 2020

Codecov Report

Merging #44720 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master   #44720   +/-   ##
=======================================
  Coverage   67.95%   67.95%           
=======================================
  Files         384      384           
  Lines       49763    49763           
=======================================
  Hits        33817    33817           
  Misses      15946    15946           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7036e91...b9445f0. Read the comment docs.

Copy link
Contributor

@facebook-github-bot facebook-github-bot left a comment

Choose a reason for hiding this comment

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

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

@facebook-github-bot
Copy link
Contributor

@ngimel merged this pull request in 07d9cc8.

@IvanYashchuk IvanYashchuk deleted the fix-issue-43141 branch September 17, 2020 06:46
xuzhao9 pushed a commit that referenced this pull request Sep 18, 2020
Summary:
Added missing error checks for the CPU version of `triangular_solve`.
Fixes #43141.

Pull Request resolved: #44720

Reviewed By: mruberry

Differential Revision: D23733400

Pulled By: ngimel

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

torch.triangular_solve gives incorrect results (CPU) for inconsistent systems and no warnings (CPU&CUDA).

5 participants