Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Release 0.26.1 #2485

Merged
merged 7 commits into from
Jul 11, 2024
Merged

Release 0.26.1 #2485

merged 7 commits into from
Jul 11, 2024

Conversation

dweindl
Copy link
Member

@dweindl dweindl commented Jul 11, 2024

No description provided.

dweindl and others added 7 commits July 2, 2024 15:15
We have a couple of -Werror options enabled when running on GHA.
However, we should also check for the repo name.
If amici is used in tests of any downstream projects, we don't want errors.

Also use CMake's `-Wno-dev` for non-amici-ci installations.

Fixes AMICI-dev#2477
Update the minimum required gcc version to provide a more actionable warning
than sth like `'reduce' is not a member of 'std'` (see e.g. AMICI-dev#2463).
Catch all exceptions in omp parallel regions.

Fixes AMICI-dev#2478.
@dweindl dweindl requested a review from a team as a code owner July 11, 2024 07:36
Copy link

codecov bot commented Jul 11, 2024

Codecov Report

Attention: Patch coverage is 47.36842% with 20 lines in your changes missing coverage. Please review.

Project coverage is 77.81%. Comparing base (b958cce) to head (86b1c59).
Report is 40 commits behind head on master.

Files with missing lines Patch % Lines
src/splinefunctions.cpp 29.41% 12 Missing ⚠️
src/sundials_linsol_wrapper.cpp 0.00% 5 Missing ⚠️
src/exception.cpp 33.33% 2 Missing ⚠️
src/rdata.cpp 80.00% 1 Missing ⚠️
Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2485      +/-   ##
==========================================
+ Coverage   77.65%   77.81%   +0.15%     
==========================================
  Files         324      324              
  Lines       20933    20941       +8     
  Branches     1462     1456       -6     
==========================================
+ Hits        16256    16295      +39     
+ Misses       4674     4643      -31     
  Partials        3        3              
Flag Coverage Δ
cpp 76.21% <47.36%> (+2.79%) ⬆️
cpp_python 34.08% <36.84%> (-0.01%) ⬇️
petab 36.63% <18.91%> (-0.02%) ⬇️
python 72.36% <42.10%> (+0.16%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
include/amici/model_dae.h 72.72% <100.00%> (ø)
include/amici/model_ode.h 100.00% <100.00%> (ø)
src/amici.cpp 81.41% <ø> (ø)
src/edata.cpp 83.07% <100.00%> (+0.92%) ⬆️
src/newton_solver.cpp 70.63% <100.00%> (ø)
src/spline.cpp 64.89% <ø> (ø)
src/rdata.cpp 95.36% <80.00%> (ø)
src/exception.cpp 69.69% <33.33%> (-9.10%) ⬇️
src/sundials_linsol_wrapper.cpp 15.31% <0.00%> (ø)
src/splinefunctions.cpp 77.59% <29.41%> (+0.14%) ⬆️

... and 5 files with indirect coverage changes

Copy link
Member

@FFroehlich FFroehlich left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@dweindl dweindl merged commit b0d79ff into AMICI-dev:master Jul 11, 2024
26 of 27 checks passed
@dweindl dweindl deleted the release_0.26.1 branch July 11, 2024 09:31
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