Skip to content

Comments

Ea building heating with ventilation and stookdagen#248

Merged
naudloomans merged 8 commits intomainfrom
EA_Building-heating-with-ventilation-and-stookdagen
Feb 18, 2026
Merged

Ea building heating with ventilation and stookdagen#248
naudloomans merged 8 commits intomainfrom
EA_Building-heating-with-ventilation-and-stookdagen

Conversation

@naudloomans
Copy link
Member

Heating updates to reduce summer space heat demand

  • increased irradiance
  • added ventilation
  • added stookdagen

@AteZenmo
Copy link
Collaborator

AteZenmo commented Feb 5, 2026

Ziet er opzich goed uit, enige is nu, de profilePointer temperature is hardcoded 24, ik zou daar voor nu ook even een parameter voor maken, desnoods even op het energyModel canvas onder de pf_ambientTemp.
Vraag me wel af, is dit iets dat al klaar is voor de main? Volgens Gillis was dit ook allemaal nog wat experimenteel (toch?).
Sws moeten we dit ook weer even testen met andere modellen als we dit naar de main willen hebben.

En ben wel benieuwd wat @GillisHommen en @Luc-Sol nu vinden van de J_EABuilding ventilation implementatie (ik heb daar ook aan mee gedacht, en ben het wel eens met de opzet, twijfel alleen aan of we dat misschien direct willen integreren met de loss berekening, ipv het als losse flow te berekenen en op te tellen bij de loss).

@naudloomans
Copy link
Member Author

Ik dacht ik ga er meteen voor, is wellicht nog niet de netste implementatie maar geeft wel accuratere resultaten dan hoe de simulatie tot nog toe was waardoor als het goed is al onze modellen zouden verbeteren.

…nd EPEX

p_tempForecastTime_h is set to 24 by default, others left at 18 hours.
Rename heat-related methods to include unit suffixes (_kW) for clarity (calculateLoss -> calculateLoss_kW, solarHeating -> solarHeating_kW, calculateAdditionalVentilationLoss -> calculateAdditionalVentilationLoss_kW) and update all call sites accordingly. Adjust building energy update to derive deltaEnergy from energyUse_kW (negated) to avoid double-counting and simplify logic. Remove obsolete p_buildingDefaultCooldownTimescale_hr field from AVGC data class and strip redundant
Copy link
Member Author

@naudloomans naudloomans left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

heel mooi

@naudloomans naudloomans merged commit 8e9978e into main Feb 18, 2026
1 of 2 checks passed
@naudloomans naudloomans deleted the EA_Building-heating-with-ventilation-and-stookdagen branch February 18, 2026 16:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants