fix(ets): Sse_ver ne doit pas être calculé sans baie intérieure vers l'ETS (#140) - #173
Open
elliot-redfroggy wants to merge 1 commit into
Conversation
…l'ETS Cf. méthode 3CL §6.3 : la surface sud équivalente véranda (Sse_ver) ne doit être calculée que s'il existe au moins une baie vitrée séparant l'espace tampon solarisé du reste du logement (type_adjacence = 10). Sans cette condition, Open3CL calculait un Sse_ver non nul même lorsque l'ETS n'avait que des baies vers l'extérieur, ce qui est incorrect. Fixes: - 6.2_surface_sud_equivalente.js (code legacy) : ajout d'un guard baiesAdjVeranda.length === 0 avant le calcul de SseVerandaj. - surface-sud-equivalente.service.js (refactorisé) : même guard via baiesVersEts.length > 0 avant le bloc de calcul Ssdj/Sstj/SseVerandaj. Tests: - Nouveau spec surface-sud-equivalente-ets-no-interior-bay.spec.js avec 3 scénarios : ETS sans baie intérieure, ETS avec baie intérieure, et absence d'ETS. Closes Open3CL#140
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
1 participant
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.
Problème
Un ETS (Espace Tampon Solarisé / véranda) comporte deux types de baies :
baie_ets)type_adjacence = 10)Selon la méthode 3CL §6.3, la surface sud équivalente véranda (
Sse_ver) ne doit être calculée que si des baies vitrées séparant le logement de l'ETS existent (type_adjacence = 10). Sans elles, les apports solaires de l'ETS ne peuvent physiquement pas pénétrer dans la partie habitable.Or Open3CL calculait
Sse_vermême en l'absence de telles baies, ce qui produit des apports solaires artificiels.DPE de reproduction :
2518E0102737H(ETS sans baie intérieure)Solution
Ajout d'un guard
baiesAdjVeranda.length === 0(ancien code6.2_surface_sud_equivalente.js) etbaiesVersEts.length > 0(nouveau service refactorisé) pour court-circuiter le calcul deSse_verlorsqu'aucune baie n'est adjacente à l'ETS côté logement.Fichiers modifiés
src/6.2_surface_sud_equivalente.js— code legacysrc/features/engine/domain/apport_et_besoin/surface-sud-equivalente.service.js— service refactorisésrc/features/engine/domain/apport_et_besoin/surface-sud-equivalente-ets-no-interior-bay.spec.js— nouveaux tests unitaires (3 scénarios)Closes #140