Clatrix not transposing V* matrix in correctly in SVD return

[tel]

Clatrix not transposing V* matrix in correctly in SVD return

[nblumoe]

@mikera thanks for raising and fixing this over here.
Should we handle this somewhere upstream, too, until it gets fixed in clatrix?

I currently have pending changes to Incanter with fixed tests. However these tests fail now, when using the clatrix implementation. Of course I could add something to Incanter, but core.matrix would seem to be more appropriate.

More importantly: Can we get this fix merged, please?

[mikera]

@nblumoe OK I merged it.... don't seem to have been any objections to this fix.

I can't make official Clatrix releases though

[tel]

Do you not have Clojars access? I can fix that.

—
Sent from Mailbox

On Mon, Jun 15, 2015 at 8:37 AM, Mike Anderson [email protected]
wrote:

@nblumoe OK I merged it.... don't seem to have been any objections to this fix.

I can't make official Clatrix releases though

Reply to this email directly or view it on GitHub:
#63 (comment)

[mikera]

@tel - actually looks like I do have membership of the clatrix group on clojars....
a) does that mean I can make a release?
b) Should I? (this isn't my project after all!)

[tel]

Ah, great!

I haven’t been an active maintainer for quite some time, so I don’t want to weigh in on decisions like that. I just wanted to make sure there wasn’t any structural reason you couldn’t release :)

Pinging @Quantisan?

—
Sent from Mailbox

On Mon, Jun 15, 2015 at 10:41 AM, Mike Anderson [email protected]
wrote:

@tel - actually looks like I do have membership of the clatrix group on clojars....
a) does that mean I can make a release?

b) Should I? (this isn't my project after all!)

Reply to this email directly or view it on GitHub:
#63 (comment)

[Quantisan]

Please do @mikera
Looks to me like you've been the maintainer for this library for a while, so you make the call.

[tel]

I'm happy discussing anything necessary for the release as necessary as well.

[mikera]

OK I created a 0.5.0 release - hope that works for everyone!

[tel]

Great! Thanks!