Skip to content

Remove ReturnData::nx, ReturnData::nxtrue#2966

Merged
dweindl merged 2 commits intoAMICI-dev:mainfrom
dweindl:nxrdata
Oct 6, 2025
Merged

Remove ReturnData::nx, ReturnData::nxtrue#2966
dweindl merged 2 commits intoAMICI-dev:mainfrom
dweindl:nxrdata

Conversation

@dweindl
Copy link
Member

@dweindl dweindl commented Oct 6, 2025

Replacements ReturnData::nx_rdata and ReturnData::nxtrue_rdata have been installed long ago. Let's get rid of the aliases.

@dweindl dweindl self-assigned this Oct 6, 2025
Replacements `ReturnData::nx_rdata` and `ReturnData::nxtrue_rdata` have been installed long ago. Let's get rid of the aliases.
@dweindl dweindl added the breaking change Issue changes API in backwards-incompatible way label Oct 6, 2025
@codecov
Copy link

codecov bot commented Oct 6, 2025

Codecov Report

❌ Patch coverage is 39.58333% with 29 lines in your changes missing coverage. Please review.
✅ Project coverage is 76.61%. Comparing base (5e0acb2) to head (777c2bb).
⚠️ Report is 122 commits behind head on main.

Files with missing lines Patch % Lines
src/rdata.cpp 72.22% 5 Missing ⚠️
models/model_calvetti_py/main.cpp 0.00% 3 Missing ⚠️
models/model_dirac_py/main.cpp 0.00% 3 Missing ⚠️
models/model_events_py/main.cpp 0.00% 3 Missing ⚠️
models/model_jakstat_adjoint_py/main.cpp 0.00% 3 Missing ⚠️
models/model_nested_events_py/main.cpp 0.00% 3 Missing ⚠️
models/model_neuron_py/main.cpp 0.00% 3 Missing ⚠️
models/model_robertson_py/main.cpp 0.00% 3 Missing ⚠️
models/model_steadystate_py/main.cpp 0.00% 3 Missing ⚠️
Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #2966      +/-   ##
==========================================
- Coverage   77.29%   76.61%   -0.69%     
==========================================
  Files         307      307              
  Lines       19912    19910       -2     
  Branches     1502     1502              
==========================================
- Hits        15391    15254     -137     
- Misses       4508     4643     +135     
  Partials       13       13              
Flag Coverage Δ
cpp 72.77% <39.58%> (-0.01%) ⬇️
cpp_python 38.29% <22.91%> (-0.01%) ⬇️
petab 39.35% <50.00%> (-0.01%) ⬇️
python 70.32% <39.58%> (-0.01%) ⬇️
sbmlsuite-jax ?

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

Files with missing lines Coverage Δ
include/amici/rdata.h 100.00% <ø> (ø)
include/amici/serialization.h 95.17% <100.00%> (ø)
python/sdist/amici/numpy.py 87.82% <ø> (ø)
src/hdf5.cpp 85.68% <100.00%> (ø)
models/model_calvetti_py/main.cpp 0.00% <0.00%> (ø)
models/model_dirac_py/main.cpp 0.00% <0.00%> (ø)
models/model_events_py/main.cpp 0.00% <0.00%> (ø)
models/model_jakstat_adjoint_py/main.cpp 0.00% <0.00%> (ø)
models/model_nested_events_py/main.cpp 0.00% <0.00%> (ø)
models/model_neuron_py/main.cpp 0.00% <0.00%> (ø)
... and 3 more

... and 6 files with indirect coverage changes

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

@dweindl dweindl marked this pull request as ready for review October 6, 2025 13:32
@dweindl dweindl requested a review from a team as a code owner October 6, 2025 13:32
@dweindl dweindl merged commit 3d46b6a into AMICI-dev:main Oct 6, 2025
16 of 18 checks passed
@dweindl dweindl deleted the nxrdata branch October 6, 2025 17:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

breaking change Issue changes API in backwards-incompatible way

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants