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

Retro updated #97

Merged
merged 11 commits into from
Apr 30, 2021
Merged

Retro updated #97

merged 11 commits into from
Apr 30, 2021

Conversation

lisazeyen
Copy link
Collaborator

update of the build_retro_cost.py script to include the before neglected losses by ventilation and thermal bridges, as well as heat gains (by solar radiation and internal heat gains) in the calculation.

Standard values and building topologies are taken from the Episcope/Tabula Project

@nworbmot
Copy link
Member

nworbmot commented Apr 18, 2021

@lisazeyen This is looking great! Sorry it took so long to review. I've tested it and it all works perfectly. Before I pull it into the master could you please make some very minor adjustments:

  • In config.yaml the switches 'retro_exogen' : False and 'retro_endogen' : True contain duplicated information don't they? They would never both be True, right? Only one or the other? You could then replace with only one, e.g. 'retro_endogen' : True.
  • In config.yaml section on retrofitting, make clear which settings apply for exogeneous and which for endogeneous retrofitting, e.g. dE is only for exogeneous, right?
  • Document changes to retrofitting module in doc/release_notes.rst.
  • Update the data documentation in doc/data.csv, i.e. remove average_surface_components.csv, add tabula-calculator-calcsetbuilding.csv.
  • Include some documentation in doc/supply_demand.rst or a new page doc/retrofitting.rst on how the retrofitting module works, even if it's just a link to the paper Mitigating heat demand peaks in buildings in a highly renewable European energy system.

…network.py script, adjust exogenously assumed space heat demand reductions to more conservative assumptions
@lisazeyen
Copy link
Collaborator Author

thanks a lot for your feed-back @nworbmot !

  • I have renamed in the config.yaml the previously called 'retro_exogen' to 'vary_space_heat_exogenously'. This factor decreases the space heat demand. This does not necessarily has to represent building renovation, but could also stand for a reduction in space heat demand due to climate change, population decrease, demolition rate and new buildings... Therefore one could also choose both switches 'vary_space_heat_exogenously' and 'retro_endogen' to be set to True. That would mean we would have a decrease in space heat demand which we get for free (e.g. because of climate change/new buildings) and we still could opimise to reduce the remaining space heat demand with building renovation
  • the former option 'dE' is now called 'vary_space_heat_exogenously_factor'. It does reduce the space heat demand before losses of district heating are applied.
    I updated the space heat demand reductions with more conservative assumptions from the EUCALC scenario: LTS baseline.
    This scenario corresponds to :

The combination of lever positions under this scenario reproduces, as far as possible, the main sectoral assumptions and key outputs of the LTS Baseline by 2050 published by the European Commission (European Commission 2018).

European Commission 2018a European long-term strategic vision for a prosperous, modern, competitive and climate neutral economy.

EUCALC has in total 16 different scenarios, it might be interesting to add them as an option to the sector-coupling model (including the CO2 emission pathways, industry assumptions, heat demand reductions, transport demand assumptions).

  • I have updated the documentation and the release notes

Maybe there is a better name for 'vary_space_heat_exogenously_factor' it is a bit long...

@nworbmot
Copy link
Member

Looks great, thank you @lisazeyen for all your hard work on this module update! I did change the name in the end, but it's not shorter :-). I'll merge now.

@nworbmot nworbmot merged commit c87b45a into master Apr 30, 2021
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.

2 participants