Skip to content
New issue

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

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

Already on GitHub? # to your account

Safe lax::vec_uninit #334

Merged
merged 2 commits into from
Sep 8, 2022
Merged

Safe lax::vec_uninit #334

merged 2 commits into from
Sep 8, 2022

Conversation

termoshtt
Copy link
Member

Split from #333

  • Create lax::alloc private submodule
  • Remove unsafe marker from lax::vec_uninit

@termoshtt termoshtt self-assigned this Sep 8, 2022
@codecov
Copy link

codecov bot commented Sep 8, 2022

Codecov Report

Merging #334 (802149b) into master (f7356c7) will increase coverage by 0.00%.
The diff coverage is 96.55%.

❗ Current head 802149b differs from pull request most recent head 4737b9c. Consider uploading reports for the commit 4737b9c to get more accurate results

@@           Coverage Diff           @@
##           master     #334   +/-   ##
=======================================
  Coverage   89.92%   89.93%           
=======================================
  Files          72       72           
  Lines        3912     3914    +2     
=======================================
+ Hits         3518     3520    +2     
  Misses        394      394           
Impacted Files Coverage Δ
lax/src/solveh.rs 75.51% <66.66%> (ø)
lax/src/eigh.rs 95.00% <83.33%> (ø)
lax/src/alloc.rs 90.90% <90.90%> (ø)
lax/src/eig.rs 100.00% <100.00%> (ø)
lax/src/layout.rs 81.15% <100.00%> (ø)
lax/src/least_squares.rs 100.00% <100.00%> (ø)
lax/src/opnorm.rs 100.00% <100.00%> (ø)
lax/src/qr.rs 100.00% <100.00%> (ø)
lax/src/rcond.rs 100.00% <100.00%> (ø)
lax/src/solve.rs 100.00% <100.00%> (ø)
... and 3 more

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@termoshtt termoshtt marked this pull request as ready for review September 8, 2022 16:00
@termoshtt termoshtt added the change Compatible change label Sep 8, 2022
@termoshtt termoshtt merged commit 87107be into master Sep 8, 2022
@termoshtt termoshtt deleted the safe-lax-vec-uninit branch September 8, 2022 16:03
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
change Compatible change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant