Skip to content

Conversation

jturner314
Copy link
Member

@jturner314 jturner314 commented May 28, 2021

This PR adds more tests for the Solve trait and fixes the implementation of solve_h_* for complex inputs with standard layout.

@jturner314 jturner314 added the bug Bug issue, or bug fix change label May 28, 2021
@jturner314 jturner314 changed the title Fix Solve::solve_h for complex inputs with standard layout Fix Solve::solve_h_* for complex inputs with standard layout May 28, 2021
@jturner314 jturner314 force-pushed the fix-solve_h-complex branch from 7f2b725 to c698704 Compare May 28, 2021 01:12
@codecov
Copy link

codecov bot commented May 28, 2021

Codecov Report

Merging #296 (ffe65cb) into master (082f01d) will increase coverage by 0.78%.
The diff coverage is 100.00%.

Impacted file tree graph

@@ Coverage Diff @@ ## master #296 +/- ## ========================================== + Coverage 89.01% 89.79% +0.78%  ========================================== Files 71 71 Lines 3578 3607 +29 ========================================== + Hits 3185 3239 +54  + Misses 393 368 -25 
Impacted Files Coverage Δ
lax/src/solve.rs 100.00% <100.00%> (ø)
ndarray-linalg/tests/solve.rs 100.00% <100.00%> (ø)
ndarray-linalg/src/solve.rs 92.74% <0.00%> (+20.16%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 082f01d...ffe65cb. Read the comment docs.

@jturner314 jturner314 force-pushed the fix-solve_h-complex branch from c698704 to ffe65cb Compare May 28, 2021 01:54
@termoshtt termoshtt merged commit 3a8f0c5 into rust-ndarray:master Jun 12, 2021
termoshtt added a commit that referenced this pull request Jun 12, 2021
@jturner314 jturner314 deleted the fix-solve_h-complex branch June 13, 2021 23:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Bug issue, or bug fix change

2 participants