Skip to content

Add seasonal/annual utilisation penalties to the appraisal mini-dispatch - #1548

Draft
tsmbland wants to merge 10 commits into
mainfrom
mini_dispatch_flatten_activity
Draft

tsmbland wants to merge 10 commits into
mainfrom
mini_dispatch_flatten_activity

Conversation

@tsmbland

@tsmbland tsmbland commented Sep 14, 2026

Copy link
Copy Markdown
Collaborator

Description

Adding seasonal/annual utilisation penalties to the appraisal mini-dispatch. These are nearly identical to the equivalent constraints used in the dispatch optimisation, but note the change of sign on the penalty as the appraisal optimisation is a MAX optimisation (whereas dispatch is MIN)

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

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

⚠️ Performance Alert ⚠️

Possible performance regression was detected for benchmark.
Benchmark result of this commit is worse than the previous benchmark result exceeding threshold 1.30.

Benchmark suite Current: f0af7c6 Previous: 6a19604 Ratio
example_run/muse1_default example 474107563.5 ns 337702386.5 ns 1.40
select_best_assets_parallel/01 13557156.78125 ns 9132896.765151516 ns 1.48
select_best_assets_parallel/05 38616796.375 ns 26819628.777777776 ns 1.44
select_best_assets_parallel/10 66001931.83333333 ns 45113674.75 ns 1.46
select_best_assets_parallel/15 95574467.25 ns 65351435.33333333 ns 1.46
select_best_assets_parallel/20 126071404.25 ns 85097739.5 ns 1.48
select_best_assets_sequential/01 13579621.07403846 ns 9107675.635526314 ns 1.49
select_best_assets_sequential/05 65227609.666666664 ns 43859325.75 ns 1.49
select_best_assets_sequential/10 129596949.5 ns 87553632.5 ns 1.48
select_best_assets_sequential/15 195143568.5 ns 131658938.5 ns 1.48
select_best_assets_sequential/20 259361474 ns 175652280.5 ns 1.48

This comment was automatically generated by workflow using github-action-benchmark.

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