stat: use the updated decomposition extraction API #170
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Will you rename Right
and Left
? There are also other candidates to follow the same pattern, and also on PC
} else if d, n := dst.Dims(); (n != 0 || d != 0) && (d != c.d || n != min(c.n, c.d)) { | ||
panic(matrix.ErrShape) | ||
if dst != nil { | ||
if d, n := dst.Dims(); (n != 0 || d != 0) && (d != c.d || n != min(c.n, c.d)) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Split this into two lines?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
:) This is an intentionally left sore to provoke the exporting of IsZero
in mat64.
} else if d, n := dst.Dims(); (n != 0 || d != 0) && (n != c.yd || d != c.xd) { | ||
panic(matrix.ErrShape) | ||
if dst != nil { | ||
if d, n := dst.Dims(); (n != 0 || d != 0) && (n != c.yd || d != c.xd) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also here?
} else if d, n := dst.Dims(); (n != 0 || d != 0) && (n != c.yd || d != c.yd) { | ||
panic(matrix.ErrShape) | ||
if dst != nil { | ||
if d, n := dst.Dims(); (n != 0 || d != 0) && (n != c.yd || d != c.yd) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
And here?
I left |
Yea, I think they should change. Consistent naming is best for everyone. |
Also propagate the `dst := x.PTo(y)` idiom to stat.
528411d
to
0b62c97
Compare
This has been done in gonum/gonum. |
Ooops. No it hasn't. |
It looks like this happened in the other repo. |
I have left the CC methods
Left
andRight
as they are rather than renaming toLeftTo
andRightTo
, which they probably should be changed to.@btracey @vladimir-ch Please take a look.
Depends on gonum/matrix#436.