[09:38:28.958456 WARN] [openstudio.energyplus.ForwardTranslator] As of 3.5.0, class boost::optional<class openstudio::IdfObject> __cdecl openstudio::energyplus::ForwardTranslator::translateTableMultiVariableLookup(class openstudio::model::TableMultiVariableLookup &) is deprecated. The entire TableMultiVariableLookup will be removed.
this PR replaces a method (getting dependent var from two independent vars) that was using TableMultiVariableLookup to a new method using TableLookup.
unit test (with vrf models) results look weird.. so I might have to look into that a bit more.
Pull Request Author
This pull request makes changes to (select all the apply):
[x] Tagged the pull request with the appropriate label (documentation, infrastructure, sampling, workflow measure, upgrade measure, reporting measure, postprocessing) to help categorize changes in the release notes.
[ ] Added tests for new measures
[x] Updated measure .xml(s)
[ ] Register values added to comstock_column_definitions.csv
[ ] Both options_lookup.tsv files updated
[ ] 10k+ test run
[ ] Change documentation written
[ ] Measure documentation written
[ ] ComStock documentation updated
[ ] Changes reflected in example .yml files
[ ] Changes reflected in README.md files
[ ] Added 'See ComStock License' language to first two lines of each code file
[ ] Implements corresponding measure tests and indexing path in test/measure_tests.txt or/and test/resource_measure_tests.txt
[ ] All new and existing tests pass the CI
Review Checklist
This will not be exhaustively relevant to every PR.
[x] Perform a code review on GitHub
[x] All related changes have been implemented: data and method additions, changes, tests
[ ] If fixing a defect, verify by running develop branch and reproducing defect, then running PR and reproducing fix
[ ] Reviewed change documentation
[x] Ensured code files contain License reference
[ ] Results differences are reasonable
[ ] Make sure the newly added measures has been added with tests and indexed properly
Pull request overview
FYI: @mdahlhausen @ChristopherCaradonna
regarding:
[09:38:28.958456 WARN] [openstudio.energyplus.ForwardTranslator] As of 3.5.0, class boost::optional<class openstudio::IdfObject> __cdecl openstudio::energyplus::ForwardTranslator::translateTableMultiVariableLookup(class openstudio::model::TableMultiVariableLookup &) is deprecated. The entire TableMultiVariableLookup will be removed.
this PR replaces a method (getting dependent var from two independent vars) that was using
TableMultiVariableLookup
to a new method usingTableLookup
.unit test (with vrf models) results look weird.. so I might have to look into that a bit more.
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.