Khalim Conn-Kowlessar daf4449d0d Optimiser objective is required on the private helpers, hoisted in repair 🟪 ... Review findings on PR #1527: - The objective is required (no sap_rating default) on _repair_to_target, _best_repair_candidate and _rescored_groups: every caller already passes it, and a default would let a future call path silently optimise SAP for a carbon/bill goal while pyright stayed green. The default stays on the public optimise_package / optimise_package_fabric_first entry points. - _best_repair_candidate hoists objective(current) out of the candidate loop: current is loop-invariant, so for the Energy-Savings bill objective this was one full BillDerivation.derive per candidate per repair iteration for the same score. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>		2026-07-10 11:17:27 +00:00
..
__init__.py	refactor(modelling): group domain/modelling into generators/scoring/optimisation	2026-06-03 13:48:36 +00:00
measure_dependency.py	refactor(modelling): type measure_type fields as MeasureType	2026-06-08 20:13:31 +00:00
optimiser.py	Optimiser objective is required on the private helpers, hoisted in repair 🟪	2026-07-10 11:17:27 +00:00