Skip to content
This repository was archived by the owner on Nov 24, 2018. It is now read-only.

Conversation

@btracey
Copy link
Member

@btracey btracey commented Aug 11, 2015

No description provided.

@btracey
Copy link
Member Author

btracey commented Aug 11, 2015

PTAL @kortschak

From what I can tell, travis is flukey. The failures have been in different releases.

@kortschak
Copy link
Member

It's odd that those were left in there.

LGTM - I've restarted the tests, so left them finish.

@kortschak
Copy link
Member

That failure on go1.5 looks like a real gc bug. We may want to try to replicate it on a local machine.

@kortschak
Copy link
Member

Passes go1.5rc1 on local machine.

@btracey
Copy link
Member Author

btracey commented Aug 12, 2015

How do you tell it's real? I've restarted the tests a couple of times on this PR and the last, and it's varied which of the releases failed. Sometimes it's 1.5rc1, sometimes it's 1.3 and 1.4, sometimes it's all of them. They all fail with a timeout (like this one does). I don't know how to distinguish GC bug from just travis being flaky.

@kortschak
Copy link
Member

kortschak commented Aug 12, 2015 via email

@btracey
Copy link
Member Author

btracey commented Aug 12, 2015

Passes on 1.5rc1 on my machine. Seems to have passed this time too. If it's a gc bug I don't know how to debug it.

btracey added a commit that referenced this pull request Aug 12, 2015
Remove unnecessary inputs to Trtrs in lapack64
@btracey btracey merged commit 9baa612 into master Aug 12, 2015
@vladimir-ch vladimir-ch deleted the fixtrtrs branch September 30, 2015 02:52
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants