Skip to content

Switched default value of i_cs_stress = 1.#3945

Merged
timothy-nunn merged 1 commit intomainfrom
3943-change-i_cs_stress-0-to-i_cs_stress-1-by-default
Nov 4, 2025
Merged

Switched default value of i_cs_stress = 1.#3945
timothy-nunn merged 1 commit intomainfrom
3943-change-i_cs_stress-0-to-i_cs_stress-1-by-default

Conversation

@grmtrkngtn
Copy link
Copy Markdown
Collaborator

Switches i_cs_stress to 1 by default as this is the default behaviour for the model.

@grmtrkngtn grmtrkngtn linked an issue Oct 21, 2025 that may be closed by this pull request
@codecov-commenter
Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 43.87%. Comparing base (e8ee138) to head (dd54344).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3945      +/-   ##
==========================================
- Coverage   46.19%   43.87%   -2.33%     
==========================================
  Files         123      123              
  Lines       28357    31883    +3526     
==========================================
+ Hits        13100    13989     +889     
- Misses      15257    17894    +2637     

☔ View full report in Codecov by Sentry.
📢 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.

Copy link
Copy Markdown
Collaborator

@timothy-nunn timothy-nunn left a comment

Choose a reason for hiding this comment

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

Happy for this change to happen. Could you get another modeller to OK this and I'll override the merge.

@grmtrkngtn grmtrkngtn requested a review from mkovari October 24, 2025 08:30
@timothy-nunn timothy-nunn merged commit b0cfe47 into main Nov 4, 2025
14 of 18 checks passed
@timothy-nunn timothy-nunn deleted the 3943-change-i_cs_stress-0-to-i_cs_stress-1-by-default branch November 4, 2025 14:07
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.

Change i_cs_stress = 0 to i_cs_stress = 1 by default

4 participants