Skip to content

Add DSP tests#289

Open
jishnub wants to merge 2 commits intoJuliaArrays:masterfrom
jishnub:dsptests
Open

Add DSP tests#289
jishnub wants to merge 2 commits intoJuliaArrays:masterfrom
jishnub:dsptests

Conversation

@jishnub
Copy link
Copy Markdown
Member

@jishnub jishnub commented Jul 29, 2023

Since conv methods may be excised from InfiniteLinearAlgebra, it's best to have tests for these here. The specialized conv methods for AbstractFill are unnecessary, and are actually less efficient than the fallback ones.

@codecov
Copy link
Copy Markdown

codecov bot commented Jul 29, 2023

Codecov Report

Merging #289 (88eddfa) into master (fef31cd) will decrease coverage by 0.01%.
Report is 1 commits behind head on master.
The diff coverage is n/a.

❗ Current head 88eddfa differs from pull request most recent head 3f8ca5c. Consider uploading reports for the commit 3f8ca5c to get more accurate results

@@            Coverage Diff             @@
##           master     #289      +/-   ##
==========================================
- Coverage   99.88%   99.88%   -0.01%     
==========================================
  Files           7        5       -2     
  Lines         871      869       -2     
==========================================
- Hits          870      868       -2     
  Misses          1        1              

see 4 files with indirect coverage changes

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

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.

1 participant