Closed JanghyunJK closed 1 month ago
@ChristopherCaradonna confirmed!
updates:
made some more changes so now the four new output variables are,
because not all coil objects have cycling ratio output var, this calculation only works for
moving on to 10k testing
merging to HPRTU PR: https://github.com/NREL/ComStock/pull/163
Pull request overview
This is coming from EUSS HP-RTU upsizing measure work (anticipated data release date = 2024 November).
When we (including @ChristopherCaradonna) initially got reviews from Paul T. and Landan T., the concern in upsizing heat pumps was mostly around increased short cycling of heat pump.
So, we want to add output variables that can capture short cycling metric. Before, we have operating COP values that can sort of give us the impact of short cycling but that is not a direct output var that only captures the impact of short cycling.
Short cycling is calculated with two values per building model:
Unitary System DX Coil Cycling Ratio
,Unitary System Sensible Cooling Rate
,Unitary System Sensible Heating Rate
.unsure if I changed all necessary stuffs to enable this.
I have not filled in everything in this file since I wasn't sure what the columns were asking:
Variable List.xlsx
I might want to see if the output vars are really added in comstock wide csv with some small comstock run.
Pull Request Author
This pull request makes changes to (select all the apply):
Author pull request checklist:
comstock_column_definitions.csv
options_lookup.tsv
files updated.yml
filesREADME.md
filestest/measure_tests.txt
or/andtest/resource_measure_tests.txt
Review Checklist
This will not be exhaustively relevant to every PR.