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

Inconsistent building types in samples and postprocessing #111

Open
JieXiong9119 opened this issue Feb 2, 2024 · 0 comments
Open

Inconsistent building types in samples and postprocessing #111

JieXiong9119 opened this issue Feb 2, 2024 · 0 comments
Assignees
Labels
bug Something isn't working

Comments

@JieXiong9119
Copy link
Collaborator

JieXiong9119 commented Feb 2, 2024

Description

Feb/2024 (EUSS 2024c1)

The office building subcategory ("LargeOffice","MediumOffice","SmallOffice") for in.comstock_building_type should follow the logic and definition here:

# Assign size category to offices
if cstock_bldg_type == 'Office':
if sqft < 25_000:
if nfloor <=3:
cstock_bldg_type = 'SmallOffice'
else:
cstock_bldg_type = 'MediumOffice'
elif sqft >= 25_000 and sqft < 150_000:
if nfloor <= 5:
cstock_bldg_type = 'MediumOffice'
else:
cstock_bldg_type = 'LargeOffice'
elif sqft >= 150_000:
cstock_bldg_type = 'LargeOffice'
else:
err_msg = f"Should never get here, check logic for {row}"
logger.error(err_msg)
raise Exception(err_msg)
assert cstock_bldg_type != 'Office' # Offices must be assigned a size

However when performing applicability check for "LargeOffice" for demand flexibility measures, it resulted in inconsistent office building types compared to metadata/samples. Here is a summary for a check performed on the comstock_baseline_metadata_and_annual_results.csv released in cycle 2:


total office building # in meta file:
74179

large office # in meta file:
4200
large office # by check:
3216
matched large office #:
2721
large office by check but not large office in meta file:
495
large office in meta file but not large office by check:
1479

medium office # in meta file:
7160
medium office # check:
7176
matched medium office #:
5791
medium office by check but not medium office in meta file:
1385
medium office in meta file but not medium office by check:
1369

small office # in meta file:
62819
small office # check:
63787
matched small office #:
62707
small office by check but not small office in meta file:
1080
small office in meta file but not small office by check:
112

Jan/2025 - Outpatient building showing Large Office in models

DF measures are now applicable to offices, schools and warehouse. However the run is showing 899 outpatient samples in this EUSS (SDR) cycle's (2024c2) results:
Image

Quick check on results shows that all these outpatient models are located in California. Suspect the error has to do with DEER building type mapping when generating the models.
I downloaded one of the outpatient models showing in DF results and indeed found conflicting building type definition within the osm model:

Image

Image

Suggestion

Please confirm the generation of in.comstock_building_type field is correct or the logic and definition in postprocessing match the source. Please check the model articulation step is correct when dealing with DEER building type

@JieXiong9119 JieXiong9119 added the bug Something isn't working label Feb 2, 2024
@JieXiong9119 JieXiong9119 changed the title Inconsistent office building types in samples and manual check Inconsistent building types in samples and postprocessing Jan 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants