6 commits

Author	SHA1	Message	Date
Khalim Conn-Kowlessar	af5dbe325d	feat(modelling): cut plan→recommendation readers onto plan_id ... Rewrite the three structurally-identical m2m-join readers (portfolio_functions.aggregate_portfolio_recommendations, Outputs.get_recommendations_from_db, export get_recommendations) to join PlanModel directly via recommendation.plan_id, dropping the plan_recommendations join and its now-unused import. The writers set plan_id (prior slice), so the rows resolve. test_export pins the export reader through the cut (its fixtures now set recommendation.plan_id). A portfolio_functions DB characterization test lands with the scenario consolidation (which provides the full-parity scenario table the aggregation writes to). Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>	2026-06-03 21:09:43 +00:00
Khalim Conn-Kowlessar	261e0769e3	created test to cover off the case of solar pv + battery	2026-03-02 17:49:58 +00:00
Khalim Conn-Kowlessar	1717e7b4c2	fixing typing issues	2026-02-26 12:45:52 +00:00
Khalim Conn-Kowlessar	3d18827961	added plan name to export	2026-02-25 13:36:02 +00:00
Khalim Conn-Kowlessar	bf3d6f4d51	working on integration test	2026-02-23 12:13:59 +00:00
Khalim Conn-Kowlessar	3e0444b3a7	working on export logic	2026-02-20 17:01:09 +00:00