Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add test for a^p where a is positive definite #22895

Merged
merged 1 commit into from
Jul 22, 2017
Merged

Add test for a^p where a is positive definite #22895

merged 1 commit into from
Jul 22, 2017

Conversation

kshyatt
Copy link
Contributor

@kshyatt kshyatt commented Jul 21, 2017

There's a special branch for this in the method for Hermitians.

@kshyatt kshyatt added linear algebra Linear algebra test This change adds or pertains to unit tests labels Jul 21, 2017
@kshyatt
Copy link
Contributor Author

kshyatt commented Jul 21, 2017

Looks like we had some build time-outs but the linalg/symmetric tests all passed. The AV failure is in compile. OK to merge? Do we want to re-run CI?

@fredrikekre
Copy link
Member

Added tests passed on all bots. Timeout on Travis and #22552 for one AV bot.

@fredrikekre fredrikekre merged commit d9903b1 into master Jul 22, 2017
@fredrikekre fredrikekre deleted the ksh/posdefpow branch July 22, 2017 01:01
jeffwong pushed a commit to jeffwong/julia that referenced this pull request Jul 24, 2017
@kshyatt @jeffwong
Add test for a^p where a is positive definite (JuliaLang#22895)
f044c63
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Sacha0 Sacha0 Sacha0 approved these changes

@fredrikekre fredrikekre fredrikekre approved these changes

Assignees
No one assigned
Labels
linear algebra Linear algebra test This change adds or pertains to unit tests
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@kshyatt @fredrikekre @Sacha0