Skip to content
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

CI: Re-enable benchmark model gradient checks / prettify gradient check output #2524

Merged
merged 1 commit into from
Oct 9, 2024

Conversation

dweindl
Copy link
Member

@dweindl dweindl commented Oct 2, 2024

@dweindl dweindl self-assigned this Oct 2, 2024
Copy link

codecov bot commented Oct 2, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 77.78%. Comparing base (e59e415) to head (b52340e).
Report is 82 commits behind head on develop.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff            @@
##           develop    #2524   +/-   ##
========================================
  Coverage    77.78%   77.78%           
========================================
  Files          325      325           
  Lines        21047    21047           
  Branches      1473     1473           
========================================
  Hits         16371    16371           
  Misses        4649     4649           
  Partials        27       27           
Flag Coverage Δ
cpp 73.49% <ø> (ø)
cpp_python 34.02% <ø> (ø)
petab 37.03% <ø> (ø)
python 72.32% <ø> (ø)

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

@dweindl dweindl force-pushed the prettify_gradcheck branch 2 times, most recently from 2673b2f to 99b6eba Compare October 8, 2024 18:35
@dweindl dweindl changed the title CI: Prettify gradient check output CI: Re-enable benchmark model gradient checks / prettify gradient check output Oct 9, 2024
@dweindl dweindl marked this pull request as ready for review October 9, 2024 08:39
@dweindl dweindl requested a review from a team as a code owner October 9, 2024 08:39
…ck output

* Make the result more informative and more readable.
* Re-enable some test problems, use stricter tolerances for some at least AMICI-dev#967
* More problem-specific settings
@dweindl dweindl force-pushed the prettify_gradcheck branch from cc547cd to b52340e Compare October 9, 2024 09:21
@dweindl dweindl merged commit 6261491 into AMICI-dev:develop Oct 9, 2024
18 of 20 checks passed
@dweindl dweindl deleted the prettify_gradcheck branch October 9, 2024 10:00
# 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