Skip to content

Conversation

johnzl-777
Copy link
Collaborator

merge_csr_mv! has the ability to add the result to the output vector that is passed in (versus just assigning values to it) but this wasn't explicitly tested by the merge_csr_mv! specific unit tests.

Unit tests now use rand instead of zeros to generate the y/input vector.

@codecov
Copy link

codecov bot commented Jan 12, 2023

Codecov Report

Merging #17 (b3bf48d) into main (f382312) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #17   +/-   ##
=======================================
  Coverage   98.55%   98.55%           
=======================================
  Files           1        1           
  Lines          69       69           
=======================================
  Hits           68       68           
  Misses          1        1           

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

@johnzl-777 johnzl-777 merged commit 0e787a0 into main Jan 12, 2023
@johnzl-777 johnzl-777 deleted the johnzl-777/in-place-tests branch January 12, 2023 21:41
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