Skip to content

Fix static arrays #376

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

Merged
merged 1 commit into from
Feb 15, 2024
Merged

Fix static arrays #376

merged 1 commit into from
Feb 15, 2024

Conversation

avik-pal
Copy link
Member

Checklist

  • Appropriate tests were added
  • Any code changes were done in a way that does not break public API
  • All documentation related to code changes were updated
  • The new code follows the
    contributor guidelines, in particular the SciML Style Guide and
    COLPRAC.
  • Any new documentation only uses public API

Additional context

Add any other context about the problem here.

Copy link

codecov bot commented Feb 15, 2024

Codecov Report

Attention: 7 lines in your changes are missing coverage. Please review.

Comparison is base (8f9a432) 58.65% compared to head (a739091) 49.51%.

Files Patch % Lines
src/internal/linear_solve.jl 46.15% 7 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #376      +/-   ##
==========================================
- Coverage   58.65%   49.51%   -9.14%     
==========================================
  Files          44       44              
  Lines        2612     2601      -11     
==========================================
- Hits         1532     1288     -244     
- Misses       1080     1313     +233     

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

@ChrisRackauckas ChrisRackauckas merged commit be573b2 into master Feb 15, 2024
@ChrisRackauckas ChrisRackauckas deleted the ap/fix_static_arrays branch February 15, 2024 17:23
# 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.

2 participants