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

Introduce QRPackedQMatrix #69

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Introduce QRPackedQMatrix #69

wants to merge 5 commits into from

Conversation

dlfivefifty
Copy link
Member

This is a version of QRPackedQ that conforms to the AbstractMatrix interface

Copy link

codecov bot commented Feb 26, 2025

Codecov Report

Attention: Patch coverage is 56.52174% with 10 lines in your changes missing coverage. Please review.

Project coverage is 92.12%. Comparing base (6874396) to head (3710276).

Files with missing lines Patch % Lines
src/qr.jl 43.75% 9 Missing ⚠️
src/MatrixFactorizations.jl 85.71% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master      #69      +/-   ##
==========================================
- Coverage   92.64%   92.12%   -0.53%     
==========================================
  Files           9        9              
  Lines        1401     1409       +8     
==========================================
  Hits         1298     1298              
- Misses        103      111       +8     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant