search

trynthink / scout

A tool for estimating the future energy use, carbon emissions, and capital and operating cost impacts of energy efficiency and demand flexibility technologies in the U.S. residential and commercial building sectors.
https://scout.energy.gov
Other
61 stars 23 forks source link

`ecm_prep.py --split_fuel --detail_brkout --health_costs` results in an error #248

Closed dewittpe closed 2 years ago

dewittpe commented 2 years ago

When running

python ecm_prep.py --split_fuel --detail_brkout --health_costs

I get the following

Importing supporting data...Complete
Initializing measures...Complete
Traceback (most recent call last):
  File "/Users/pdewitt/scout/ecm_prep.py", line 12285, in <module>
    main(base_dir)
  File "/Users/pdewitt/scout/ecm_prep.py", line 12043, in main
    meas_prepped_objs = prepare_measures(
  File "/Users/pdewitt/scout/ecm_prep.py", line 10850, in prepare_measures
    [m.fill_mkts(
  File "/Users/pdewitt/scout/ecm_prep.py", line 10850, in <listcomp>
    [m.fill_mkts(
  File "/Users/pdewitt/scout/ecm_prep.py", line 2372, in fill_mkts
    self.check_mkt_inputs()
  File "/Users/pdewitt/scout/ecm_prep.py", line 7747, in check_mkt_inputs
    raise ValueError(
ValueError: Input names in the following list are invalid for ECM 'Prospective Commercial CCHP': ['AIA_CZ1', 'AIA_CZ2']

The error follows several warning messages about for things that will not be prepared, but I think that is expected behavior.

UserWarning: WARNING: Best Commercial Gas Boiler does not apply to the electric fuel type; versions of this ECM with low/high public health cost adders will not be prepared.

The error is generated with running the code from commit 2950a3bf06f0ec94a197cd0eed02b11af0e68750

jtlangevin commented 2 years ago

@dewittpe --health_costs will only work when running with EMM regions (via the --alt_regionsoption) as described in the docs on that option here.

When you run with a non-EMM region option (AIA, in this case), the routine will automatically switch to running with EMM regions, but it should warn you about it (in the code, here). Given the switch, any ECMs with non-EMM region settings (like the CCHP one you referenced) will throw that error about invalid names.

dewittpe commented 2 years ago

@jtlangevin thank you for the link to the documentation.