Repository navigation
fix(ventilation): recalculer q4pa_conv sans reprendre la ligne du DPE - #323
Closed
elliot-redfroggy wants to merge 1 commit into
Closed
elliot-redfroggy wants to merge 1 commit into
elliot-redfroggy wants to merge 1 commit into
Conversation
Les DPE soumis par un client sont contrôlés en recalculant avec le moteur : un écart de plus de 5 % avec le DPE entraîne un rejet. Certains logiciels (LICIEL notamment) retiennent une ligne q4pa_conv qui contredit l'isolation déclarée des parois ou les joints des menuiseries. En reprenant cette ligne (bug-for-bug), le moteur reproduisait l'erreur et ces DPE passaient le contrôle. La ligne tv_q4pa_conv_id du DPE n'est plus reprise : q4pa_conv est toujours recalculé selon la méthode à partir des données d'entrée. Une ligne contradictoire est seulement signalée dans console.error, comme avant #312. - retrait de la reprise de la ligne « surfaces isolées » (#312, c090c47 / b16eaca) ; - retrait de la reprise des joints quand presence_joint n'est pas exporté (#294, 0e8b1a2), en legacy comme dans la nouvelle architecture (getQ4paConvById et indicateur presenceJointNonRenseignee supprimés). 2287E1201982M : q4pa_conv 2 -> 4,6 (valeur de la méthode, parois non isolées).
michaeldesigaud
self-requested a review
October 28, 2026 10:21
Collaborator
Author
|
Remplacée par #324 (reprise conservée sous compat_for_corpus_analysis). |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Contexte
Un client contrôle les DPE qui lui sont soumis en les recalculant avec le moteur : si le résultat diffère de plus de 5 % du DPE, celui-ci est rejeté à l'entrée.
Certains logiciels (LICIEL notamment) retiennent une ligne
q4pa_conv(tv_q4pa_conv_id) qui contredit l'isolation déclarée des murs/plafonds ou la présence de joints des menuiseries. Depuis #312 et #294, le moteur reprenait cette ligne (compatibilité bug-for-bug) : il reproduisait donc l'erreur du logiciel, et ces DPE passaient le contrôle des 5 %. Le client souhaite que le moteur recalculeq4pa_convselon la méthode afin que ces DPE soient signalés.Changement
q4pa_convest toujours recalculé à partir desdonnee_entree(isolation des parois, joints des menuiseries). La ligne du DPE n'est plus reprise ; une ligne contradictoire est seulement signalée parconsole.error, comme avant #312.src/4_ventilation.js(tv_q4pa_conv) : retrait de la reprise de la ligne « surfaces isolées » (Renouvellement d'air : ligne q4pa_conv du DPE incohérente avec l'isolation des parois (2287E1201982M, +12 % ep) #312, c090c47 / b16eaca) et de la reprise des joints quandpresence_jointn'est pas exporté (Renouvellement d'air : presence_joint absent sur toutes les menuiseries — reprendre la valeur du DPE (q4pa 2,5) en bug_for_bug #294, 0e8b1a2). Les helpers devenus inutiles (presenceJointNonRenseignee,isLigneQ4paMemePeriode,isLigneQ4paSurfacesIsolees) sont supprimés.DeperditionVentilationService.q4paConv, ainsi que de la plomberie associée (DeperditionEnveloppeService.#presenceJointNonRenseignee,TvStore.getQ4paConvById).ratioSurfaceJoint >= 0.5est conservé.Impact corpus (89 972 DPE,
calcul_3clpar défaut, main f9ac5d9 → branche)ok= les 4 postes (conso_ch, conso_ecs, ep_conso_5_usages ou _m2, emission_ges_5_usages ou _m2) sont à ≤ 5 % de la valeur ADEME (avant 2026 : comparaison aux sortiescoeff_2_3_*). 51 DPE sont en erreur sur les deux versions ; ils sont exclus.OK → KO par logiciel : LICIEL 205, puis des versions non identifiées par nom (3.0.5 : 20, 3.1.1 : 17, 3.0.4 : 13, 3.0.3 : 11, 2.0.4 : 11…), DPEWIN 6, Pleiades 6.
Principales transitions de q4pa_conv : 2,5 → 4,6 (295), 2 → 4,6 (213), 1,9 → 2 (197).
La baisse de conformité est attendue : ces DPE sont ceux dont la ligne q4pa_conv contredit leurs données d'entrée, c'est-à-dire ceux que le client veut détecter.
Tests
src/4_ventilation.spec.js: les tests de reprise vérifient désormais qu'une ligne du DPE contradictoire donne la valeur recalculée ; couverture 100 % sursrc/4_ventilation.js.deperdition-ventilation.service.spec.js,deperdition-enveloppe.service.spec.js,tv.store.spec.js: mis à jour.test/open3cl-q4pa-ligne-dpe.spec.js: 2287E1201982M → q4pa_conv recalculé à 4,6 (ligne 1) au lieu de 2 ; le scénario 2469E2650374G avec murs isolés reste à 2 / classe E.