Skip to content

DiseasyImmunity: Update default optimisers - #289

Draft
RasmusSkytte wants to merge 31 commits into
mainfrom
feature/more-optimiser-script-changes
Draft

DiseasyImmunity: Update default optimisers#289
RasmusSkytte wants to merge 31 commits into
mainfrom
feature/more-optimiser-script-changes

Conversation

@RasmusSkytte

@RasmusSkytte RasmusSkytte commented Aug 25, 2026

Copy link
Copy Markdown
Contributor

Intent

This PR introduces a few more quality of life updates to the optimiser script and updates the default optimisers in DiseasyImmunity based on the results of a fresh re-run of optimiser script.

This new run also include a mixed penalty mode which the vignette is updated to display. That is, we test a scenario where monotonous = TRUE while individual_level = FALSE

Approach

N/A

Known issues

N/A

Checklist

  • The PR passes all local unit tests
  • I have documented any new features introduced
  • If the PR adds a new feature, please add an entry in NEWS.md
  • A reviewer is assigned to this PR

@RasmusSkytte
RasmusSkytte force-pushed the feature/more-optimiser-script-changes branch from 8662908 to 2bbfb92 Compare August 25, 2026 08:15
@codecov

codecov Bot commented Aug 25, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 67.19%. Comparing base (62c0545) to head (7edf802).

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #289      +/-   ##
==========================================
+ Coverage   67.16%   67.19%   +0.02%     
==========================================
  Files          28       28              
  Lines        4648     4648              
==========================================
+ Hits         3122     3123       +1     
+ Misses       1526     1525       -1     

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

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

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.

1 participant