Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Exclude wrong DPE from corpus #42

Open
wants to merge 6 commits into
base: main
Choose a base branch
from
Open

Conversation

fperinel
Copy link
Collaborator

@fperinel fperinel commented Jun 7, 2024

Close #36
Close #38
Close #39
Close #40
Close #41
Close #43

@jzck
Copy link
Owner

jzck commented Jun 8, 2024

Alors la ca devient intéressant,

Pour 2187E0982013C, la déviance est facile a trouver, en fait ce logiciel utilise le champ paroi_ancienne au lieu de enduit_isolant_paroi_ancienne. Si on utilise le bon champ on retombe sur la meme valeur qu'eux. (cf https://github.com/jzck/Open3CL/blob/main/src/3.2.1_mur.js#L72)

         {
           donnee_entree: {
+            reference: "mur_0"
+            tv_coef_reduction_deperdition_id: 1
             description: "Mur Nord, Sud, Est, Ouest - Mur en pierre de taille et moellons avec remplissage tout>
             enum_materiaux_structure_mur_id: "3"
             enum_methode_saisie_u0_id: "2"
             enum_methode_saisie_u_id: "1"
             enum_orientation_id: "4"
             enum_type_adjacence_id: "1"
             enum_type_doublage_id: "2"
             enum_type_isolation_id: "2"
             paroi_ancienne: 1
             surface_paroi_opaque: 134.76
             surface_paroi_totale: 134.76
             tv_umur0_id: 15
           }
           donnee_intermediaire: {
             b: 1
-            umur: 0.81545
+            umur: 1.9
-            umur0: 0.81545
+            umur0: 1.9
           }
         }

On a 1.9 car on ne prends pas en compte l'enduit isolant car ils utilisent un mauvais nom de champ, si on fait 1/(1/1.9)+0.7 on tombe bien sur 0.81545.

Le probleme avec ca, c'est que certains DPE set le flag paroi_ancienne et font le calcul avec le facteur 0.7, comme on le voit avec 2187E0982013C. Alors que certains DPE set aussi le flag mais n'appliquent pas le facteur 0.7 (exemple 2287E1923356Q).

L'idéal pour moi, c'est de reproduire le calcul que fait l'autre logiciel.

L'intéret c'est qu'en reproduisant un bug sur le mur, on peut quand meme continuer la suite de test sur les autres elements du dpe, chauffage, ecs, etc... Alors que si l'on exclut entierement le DPE, ca veut parfois dire d'exclure tous les DPE produit par un des logiciel, donc une suite de test avec des oeillères.

Un deuxieme intéret, si on n'exclut pas de dpe et qu'on trouve les endroits ou l'on est pas d'accord, c'est de pouvoir repertorier les DPE qui sont faux, où exactement ils sont faux, pourquoi ils sont faux, et quelle est la différence de resultat avec le bon calcul (changement de lettre etc,). C'est un sujet hyper intéressant dans la mesure où les DPE sont utilisés légalements pour des interdictions de louer, des ventes, etc...

Qu'en penses-tu?

@michaeldesigaud michaeldesigaud requested a review from jzck June 20, 2024 13:50
@jzck jzck force-pushed the main branch 3 times, most recently from 79d1729 to 08355b4 Compare August 5, 2024 16:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants