Skip to content

fix(chauffage): cascade avec priorité, besoin affecté au générateur principal - #237

Merged
michaeldesigaud merged 1 commit into
mainfrom
fix/cascade-avec-priorite
Oct 4, 2026
Merged

michaeldesigaud merged 1 commit into
mainfrom
fix/cascade-avec-priorite

Conversation

@elliot-redfroggy

Copy link
Copy Markdown
Collaborator

Problème

§13.2.1.3 : en cascade avec priorité, le générateur principal couvre le besoin (taux de charge sur sa seule Pn) et le générateur en relève a Cch = 0 dans Tribu. Open3CL ignorait priorite_generateur_cascade (1 = principal, 2 = secondaire ; sans priorité, tous principaux).

Validation Tribu (avec K propane + correction adaptateur)

Cas Tribu Avant Après
APP5-0-4 10 893,38 13 443,0 (+23,4 %) 10 893,38
APP5-0-41 11 039,69 +15,6 % 11 039,69
APP5-0-43 13 608,56 −1,2 % 13 608,57

⚠️ À valider : vérifié uniquement avec 2 générateurs, priorité 1/2, configuration simple. Les 62 fixtures ADEME en cascade ont toutes la priorité 1 → le cas « 2 = relève » n'est pas validable sur données ADEME. Auxiliaires du générateur en relève (besoin nul) à confirmer.

Autotests ADEME (Tribu), base 1.8.21

Grandeurs conformes Cch Cas 100 %
1.8.21 92,0 % 63/82 60/82
3 PR chauffage (Pveil, K propane, cascade priorité) 94,7 % 70/82 67/82
+ 2 corrections adaptateur autotests 97,3 % 77/82 73/82

Aucune bascule conforme → non conforme. Fixtures ADEME (296, conso_ch ±1 %) : 192 conformes avant et après.
Tests : ajouts uniquement, les nouveaux tests échouent sans le correctif, couverture 100 % du fichier modifié, eslint/prettier OK.

… tout le besoin

13.2.1.3 : avec priorite_generateur_cascade (1 principal, 2 relève), seuls les
générateurs de plus petite priorité reçoivent le besoin et dimensionnent Pnominal
(cdimref). Sans priorité ou priorités égales : prorata des Pn inchangé.
Reproduit Tribu sur les autotests État APP5-0-4/41/43.
@michaeldesigaud
michaeldesigaud self-requested a review October 4, 2026 11:33
@michaeldesigaud michaeldesigaud added bug Something isn't working auto_tests_cstb labels Oct 4, 2026
@michaeldesigaud
michaeldesigaud merged commit b4d372b into main Oct 4, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto_tests_cstb bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants