search

IndEcol / RECC-ODYM

The RECC model
MIT License
21 stars 10 forks source link

Buildings energy demand, 3_EI_Products_UsePhase_resbuildings_t #41

Closed CarrerF closed 1 year ago

CarrerF commented 1 year ago

In about lines 2394-2398, In the new update, historical cohorts'energy intensity is multiplied by RECC_System.ParameterDict['3_SHA_EnergyCarrierSplit_Buildings']. However, this multiplication occurred already in line 1577. As consequence, hisotric energy intensity is set to 0:

image

I commented those lines and replaced with a new version of the computations without the EnergyCarrierSplit. Results seem correct, but it should be checked if computations work as intended. image

Same for nrb, but I have not fixed it yet.

stefanpauliuk commented 1 year ago

It definitely needs to look like in the Figure below, which it did before in RECC v2.4. I will double-check the entire energy calculation for historic and future age-cohorts.

stefanpauliuk commented 1 year ago

Fixed during latest RECC model update for res. and non-res buldings.