remove always-added reporting measure from Comstock

[eringold]

SimulationOutputReport was being added to the comstock workflow by default, as well as when specified in the yml. This removes the default addition: SimulationOutputReport must be specified in the yml to be included in the workflow.

Eyeballing from my recent profiling of a 10k run shows the second instance of SimulationOutputReport taking roughly 10% of the total workflow runtime.

[github-actions]

File	Coverage
All files	84%	✅
base.py	89%	✅
eagle.py	79%	✅
exc.py	57%	✅
local.py	50%	✅
postprocessing.py	84%	✅
utils.py	96%	✅
sampler/base.py	79%	✅
sampler/downselect.py	33%	✅
sampler/precomputed.py	93%	✅
sampler/residential_quota.py	61%	✅
test/test_docker.py	33%	✅
test/test_validation.py	97%	✅
workflow_generator/base.py	90%	✅
workflow_generator/commercial.py	53%	✅
workflow_generator/residential_hpxml.py	84%	✅

Minimum allowed coverage is 33%

Generated by 🐒 cobertura-action against 9cc96c5

[nmerket]

Looks pretty good. Could you put something in the changelog for this?

[eringold]

Thanks @nmerket, I've updated the changelog.