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

Allow using RDataReporting::residuals with ASA #2626

Closed
wants to merge 1 commit into from

Conversation

dweindl
Copy link
Member

@dweindl dweindl commented Dec 12, 2024

So far, it is not supported to use RDataReporting::residuals with ASA. However, there doesn't seem to be a good reason.
It's only that the respective sensitivities for y/sigmay/res won't be computed in this case.

Closes #2625

Copy link

codecov bot commented Dec 12, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 77.01%. Comparing base (2d58f54) to head (48bcb36).

Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff             @@
##           develop    #2626      +/-   ##
===========================================
- Coverage    77.05%   77.01%   -0.04%     
===========================================
  Files          332      332              
  Lines        22957    22949       -8     
  Branches      1477     1475       -2     
===========================================
- Hits         17689    17675      -14     
- Misses        5257     5263       +6     
  Partials        11       11              
Flag Coverage Δ
cpp 73.94% <ø> (-0.04%) ⬇️
cpp_python 33.49% <ø> (+<0.01%) ⬆️
petab 38.37% <ø> (+0.01%) ⬆️
python 72.10% <ø> (-0.03%) ⬇️

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

Files with missing lines Coverage Δ
src/solver.cpp 78.30% <ø> (-0.20%) ⬇️

... and 3 files with indirect coverage changes

So far, it is not supported to use `RDataReporting::residuals` with ASA.
However, there doesn't seem to be a good reason.
It's only that the respective sensitivities for y/sigmay/res won't be computed in this case.
@dweindl
Copy link
Member Author

dweindl commented Dec 16, 2024

#2625 (comment)

@dweindl dweindl closed this Dec 16, 2024
@dweindl dweindl deleted the 2625_asa_res branch December 16, 2024 07:13
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