Calcul de la déperdition des planchers bas non conforme avec 2187E0982591I

[michaeldesigaud]

Describe the bug
Dans le DPE 2187E0982591I, la déperdition des planchers bas n'est pas cohérente entre le résultat de la lib et la valeur du DPE d'origine.
Les données intermédiaires ne sont pas les mêmes.

Dans le DPE d'origine

{
"donnee_entree": {
"description": "Plancher 1 - Plancher avec ou sans remplissage non isolé donnant sur un terre-plein",
"enum_type_adjacence_id": "5", // terre-plein
"surface_paroi_opaque": 44.8,
"tv_upb0_id": 2,
"enum_type_plancher_bas_id": "2", // Plancher avec ou sans remplissage
"enum_methode_saisie_u0_id": "2", // déterminé selon le matériau et épaisseur à partir de la table de valeur forfaitaire
"enum_type_isolation_id": "2", // Non isolé
"enum_methode_saisie_u_id": "1", // Non isolé
"calcul_ue": 1,
"perimetre_ue": 12.8,
"ue": 0.38538462
},
"donnee_intermediaire": {
"b": 1,
"upb": 1.45,
"upb_final": 0.38538462,
"upb0": 1.45
}
}
En sortie de calcul on obtient :

{
"donnee_entree": {
"description": "Plancher 1 - Plancher avec ou sans remplissage non isolé donnant sur un terre-plein",
"enum_type_adjacence_id": "5",
"surface_paroi_opaque": 44.8,
"tv_upb0_id": 2,
"enum_type_plancher_bas_id": "2",
"enum_methode_saisie_u0_id": "2",
"enum_type_isolation_id": "2",
"enum_methode_saisie_u_id": "1",
"calcul_ue": 1,
"perimetre_ue": 12.8,
"ue": 0.3361538461538462,
"reference": "plancher_bas_0",
"tv_coef_reduction_deperdition_id": 5
},
"donnee_intermediaire": {
"b": 1,
"upb0": 1.45,
"upb": 1.45,
"upb_final": 0.3361538461538462
}
}
To Reproduce

Surface sur terre plein
P = 12.8
S = 44.8
2S/P = (2 * 44.8) / 12.8 = 7 (arrondi à l'entier le plus proche)
upb0 = 1.45
upb = 1.45
image

upbInf = 0.85
upbSup = 1.5
ueInf = 0.33
ueSup = 0.39

D'après la norme :

Les données ne figurant pas dans le tableau peuvent être obtenues par interpolation et extrapolation en traçant des
droites entre les valeurs les plus proches présentes dans le tableau

ue = ueInf + ( ueSup - ueInf ) * (upb - upbInf) / (upbSup - upbInf)
ue = 0.33 + ( 0.39 - 0.33 ) * (1.45 - 0.85) / ( 1.5 - 0.85)
ue = 0,385384615
upb_final = ue = 0,385384615
Expected behavior
Le calcul de la lib n'est pas bon et devrait retourner upb_final = ue = 0,385384615
Le problème semble venir de la fonction tv_ue qui fait la somme des surfaces de tous les planches sans distinction du type d'adjacence. Les planchers donnant sur des terre plein sont ajouté à ceux donnant sur vide sanitaire alors que les règles ne sont pas les mêmes.

Additional context

NodeJS version: 18.16.0

[michaeldesigaud]

@jungoliv je veux bien ton avis sur ce bug. Le pb cest que dans la lib pour un plancher bas donné:

• On lit la surface Ue et le périmètre Ue et on additionne pour tous les planchers bas (en prenant dans la liste des planchers bas uniquement ceux qui ont la meme adjacence que le plancher bas courant)
• Ca donne effectivement une mauvaise valeur comme indiqué dans ce ticket.

Ce qui m'échappe cest pourquoi dans la lib on itère sur les planchers bas pour chaque plancher bas.
Je ne vois rien de tel dans la spec.

J'ai ce commentaire associé:

/**
   * La surface Ue est la surface de tous les planchers bas ayant le même type d'adjacence
   * Le périmètre Ue est le périmètre de tous les planchers bas ayant le même type d'adjacence
   */

Si je calcul simplement le périmètre et la surface pour le plancher courant (sans faire la somme des autres plancher) je retombe sur les bonnes valeurs et jai meme une diszaine de DPE qui passent désormais dans les corpus.

Tu me confirmes ?

[jungoliv]

@michaeldesigaud la spec est un peu floue à cause des ces phrases là :

Le problème c'est que je n'arrive pas à trouver des DPE avec plusieurs planchers bas qui auraient le même type d'adjacence. Dans le DPE que tu mentionnes on voit bien qu'il y a un ue différent pour les deux planchers bas sur terre-plein. Et physiquement ça pourrait s'expliquer, par exemple une extension où l'on aurait coulé une autre dalle.

Est ce que tu as une vision de l'amélioration sur les corpus logements individuels si :
_ On calcule ue pour chaque plancher bas concerné (donc même si deux planchers bas donnent sur un même type d'adjacence on aura deux ue différents si upb, Spb et périmètre_ue sont différents
_ Si on applique cette correction par rapport à l'implémentation actuelle où l'on "fusionne" les planchers bas avec même adjacence UNIQUEMENT aux maisons individuelles

[michaeldesigaud]

@jungoliv merci pour ton retour, c'est pas le cas du DPE 2187E0982591I mentionné dans ce ticket ? il a 3 planchers bas dont deux (plancher 1 et 3) avec la meme adjacence non (enum_type_adjacence_id=5) ?

[jungoliv]

@michaeldesigaud si c'est bien le cas de ce DPE là mais je voulais en voir d'autres car je suis quasi sûr que j'avais eu la discussion avec Jérôme et que c'est lui qui m'avait montré des DPE où le calcul ue était bien fait sur du fusionné. Je me dis que c'est peut être uniquement dans le cas de l'immeuble, mais sans DPE pour vérifier c'est compliqué.

@michaeldesigaud edit : DPE 2362E1093836C qui fonctionne en fusionnant les planchers... et qui est un DPE maison... La question est de savoir si 2362E1093836C est une exception sur laquelle on a fondé nos calculs

[jungoliv]

@michaeldesigaud
Difficile de conclure sur ces DPE, par contre a priori en 2025 (et un peu avant) on trouve une variable qui s'appelle surface_ue et qui correspond à celle qui doit être utilisée dans la formule 2S/P.

Si cette variable n'est pas présente, je vois deux possibilités :
_ On teste la méthode un par un pour ue ou en sommant par adjacence et on choisi la solution qui colle s'il en y en a une, sinon on utilise une méthode par défaut (la plus performante vis à vis du corpus) et on créé la variable surface_ue correspondante
_ On opte pour une des deux méthodes en amont (la plus performante vis à vis du corpus), on créé la variable surface_ue correspondante et si le dpe ne passe pas on l'analysera unitairement.

As-tu la visibilité sur quelle méthode à les meilleurs résultats ?

[michaeldesigaud]

@jungoliv merci pour les infos. Non pas de visibilité encore je vais faire des tests et je reviens vers toi