Skip to content

Increase annual_utilisation_penalty - #1550

Merged
tsmbland merged 2 commits into
mainfrom
increase_annual_penalty
Sep 21, 2026
Merged

tsmbland merged 2 commits into
mainfrom
increase_annual_penalty

Conversation

@tsmbland

@tsmbland tsmbland commented Sep 16, 2026 •

Copy link
Copy Markdown
Collaborator

Description

Found that for #1548 I got different results on different platforms if I didn't increase the annual penalty. Doing it here in isolation to separate the effects of this from #1548

Fixes # (issue)

Type of change

  • Bug fix (non-breaking change to fix an issue)
  • New feature (non-breaking change to add functionality)
  • Refactoring (non-breaking, non-functional change to improve maintainability)
  • Optimization (non-breaking change to speed up the code)
  • Breaking change (whatever its nature)
  • Documentation (improve or add documentation)

Key checklist

  • All tests pass: $ cargo test
  • The documentation builds and looks OK: $ cargo doc
  • Update release notes for the latest release if this PR adds a new feature or fixes a bug
    present in the previous release

Further checks

  • Code is commented, particularly in hard-to-understand areas
  • Tests added that prove fix is effective or that feature works

@tsmbland
tsmbland changed the base branch from main to demand_map_v2 September 16, 2026 10:08
@codecov

codecov Bot commented Sep 16, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 90.83%. Comparing base (1a78d47) to head (22fbf4e).

Additional details and impacted files
@@              Coverage Diff               @@
##           demand_map_v2    #1550   +/-   ##
==============================================
  Coverage          90.83%   90.83%           
==============================================
  Files                 61       61           
  Lines               9019     9019           
  Branches            9019     9019           
==============================================
  Hits                8192     8192           
  Misses               508      508           
  Partials             319      319           

☔ 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.

@tsmbland
tsmbland marked this pull request as ready for review September 16, 2026 10:15
Base automatically changed from demand_map_v2 to main September 21, 2026 13:15
@tsmbland
tsmbland merged commit 9a3d2a6 into main Sep 21, 2026
9 checks passed
@tsmbland
tsmbland deleted the increase_annual_penalty branch September 21, 2026 13:16
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