Skip to content

move sub_factorize from MultivariateOrthogonalPolynomials.jl - #230

Open
dlfivefifty wants to merge 5 commits into
masterfrom
sub_factorize-for-blocks
Open

move sub_factorize from MultivariateOrthogonalPolynomials.jl#230
dlfivefifty wants to merge 5 commits into
masterfrom
sub_factorize-for-blocks

Conversation

@dlfivefifty

Copy link
Copy Markdown
Member

No description provided.

@codecov

codecov Bot commented Aug 26, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 75.00000% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 93.55%. Comparing base (7f1e4af) to head (736c5a1).

Files with missing lines Patch % Lines
src/bases/bases.jl 75.00% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #230      +/-   ##
==========================================
+ Coverage   93.52%   93.55%   +0.03%     
==========================================
  Files          12       12              
  Lines        1019     1024       +5     
==========================================
+ Hits          953      958       +5     
  Misses         66       66              

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@dlfivefifty

Copy link
Copy Markdown
Member Author

@copilot increase coverage

Co-authored-by: dlfivefifty <1229737+dlfivefifty@users.noreply.github.com>

Copilot AI commented Aug 27, 2026

Copy link
Copy Markdown

@copilot increase coverage

Added coverage for:

  • ProjectionFactorization \ AbstractQuasiMatrix (the multi-RHS backslash) in the spline transform tests
  • _sub_factorize(::Tuple{Any,Any}, ...) dispatches (both OneTo and general jr) by introducing an InfChebyshev infinite-dimensional basis in the Chebyshev tests

Commit: increase coverage for sub_factorize and ProjectionFactorization

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants