Skip to content

Conversation

@wschin
Copy link
Contributor

@wschin wschin commented Feb 5, 2021

No description provided.

@wschin wschin added the type:bug label Feb 5, 2021
@wschin wschin requested a review from SherlockNoMad February 5, 2021 06:29
@wschin wschin requested a review from a team as a code owner February 5, 2021 06:29
@wschin wschin changed the title Throw for non-supported cases Generate shape-independent graph if any input dimension < 2 Feb 5, 2021

if (A_has_shape && B_has_shape && Y_has_shape) {
if (A_has_shape && B_has_shape && Y_has_shape &&
A_shape.size() >= 2 && B_shape.size() >= 2) {
Copy link
Contributor

Choose a reason for hiding this comment

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

Matmul schema only allow inputs with dim>=2.
I thought this is an implicit prerequisite? Are you seeing inputs with less then 2 dim?

Copy link
Contributor

@SherlockNoMad SherlockNoMad Feb 5, 2021

Choose a reason for hiding this comment

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

I saw the discussion on team's channel, and understand now this is caused by a torch bug/"feature".
This fix is harmless.

Copy link
Member

Choose a reason for hiding this comment

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

Out MatMul allows vectors.


In reply to: 571201490 [](ancestors = 571201490)

@snnn
Copy link
Contributor

snnn commented Feb 10, 2021

/azp run Windows CPU CI Pipeline

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@wschin wschin closed this Feb 10, 2021
@wschin wschin reopened this Feb 10, 2021
@wschin wschin merged commit 8972621 into master Feb 10, 2021
@wschin wschin deleted the wechi/throw-matmul-grad branch February 10, 2021 07:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants