EMS and unit test fixes

NREL · Dec 21, 2023 · faeba62 · faeba62 · nrel-bot-2b · Dec 21, 2023
1 parent 8c36ff6
commit faeba62
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/src/EnergyPlus/EMSManager.cc b/src/EnergyPlus/EMSManager.cc
@@ -432,7 +432,7 @@ namespace EMSManager {
         // Update sensors with current data
         for (int SensorNum = 1; SensorNum <= state.dataRuntimeLang->NumSensors; ++SensorNum) {
             int ErlVariableNum = state.dataRuntimeLang->Sensor(SensorNum).VariableNum;
-            if ((ErlVariableNum > 0) && (state.dataRuntimeLang->Sensor(SensorNum).Index > 0)) {
+            if ((ErlVariableNum > 0) && (state.dataRuntimeLang->Sensor(SensorNum).Index > -1)) {
                 if (state.dataRuntimeLang->Sensor(SensorNum).SchedNum == 0) { // not a schedule so get from output processor
 
                     state.dataRuntimeLang->ErlVariable(ErlVariableNum).Value = RuntimeLanguageProcessor::SetErlValueNumber(

diff --git a/tst/EnergyPlus/unit/OutputReportTabular.unit.cc b/tst/EnergyPlus/unit/OutputReportTabular.unit.cc
@@ -10064,6 +10064,7 @@ TEST_F(SQLiteFixture, OutputReportTabularMonthly_CurlyBraces)
     ASSERT_TRUE(process_idf(idf_objects));
 
     Meter *meter = new Meter("Electricity:Facility");
+    meter->units = Constant::Units::J;
     state->dataOutputProcessor->meters.push_back(meter);
     state->dataOutputProcessor->meterMap.insert_or_assign("ELECTRICITY:FACILITY", (int)state->dataOutputProcessor->meters.size()-1);
     // We do need to trick it into thinking it's a weather simulation, otherwise the monthly reports aren't reported