Skip to content

fix(chauffage): conversion PCS/PCI de la veilleuse dans le rendement de génération - #235

Open
elliot-redfroggy wants to merge 1 commit into
mainfrom
fix/rg-pveil-pcs
Open

elliot-redfroggy wants to merge 1 commit into
mainfrom
fix/rg-pveil-pcs

Conversation

@elliot-redfroggy

Copy link
Copy Markdown
Collaborator

Problème

§13.2.1.2 : rg_pcs = Pmfou / (Pmcons + 0,45·QP0 + Pveil), puis rg = k·rg_pcs. QP0 est déjà converti (#205) mais pas Pveil. Tribu applique aussi k à Pveil.

Validation Tribu

  • MI4-0-1 : Rg Tribu 0,7374941696 / avant 0,7406078667 / après 0,7374941696 (≈1e-15).
  • MI4-0-1/11/12/13 : Cch −0,42 % → 0,00 % → 4 cas 100 % conformes.

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.

…ndement de génération

Comme QP0 (#205), Pveil intervient dans rg_pcs = Pmfou / (Pmcons + 0.45 QP0 + Pveil)
et doit être multipliée par k (PCS/PCI). Reproduit exactement le Rg Tribu des autotests
État MI4-0-1/11/12/13 (0.7374941696 au lieu de 0.7406078667, Cch -0.42 % -> 0.00 %).
@michaeldesigaud
michaeldesigaud self-requested a review October 4, 2026 11:32
@michaeldesigaud michaeldesigaud added bug Something isn't working auto_tests_cstb labels Oct 4, 2026

This branch has not been deployed

No deployments
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