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.

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

Fix Solve::solve_h_* for complex inputs with standard layout #296

Merged
merged 2 commits into from
Jun 12, 2021

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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants