Skip to content

Commit

Permalink
Fix a few more diffs
Browse files Browse the repository at this point in the history
  • Loading branch information
amirroth committed Dec 10, 2023
1 parent 6c53c72 commit 9ce5b6a
Showing 1 changed file with 3 additions and 0 deletions.
3 changes: 3 additions & 0 deletions src/EnergyPlus/OutputProcessor.cc
Original file line number Diff line number Diff line change
Expand Up @@ -1193,6 +1193,8 @@ namespace OutputProcessor {
ShowContinueError(state, format("..on {}=\"{}\".", ipsc->cCurrentModuleObject, ipsc->cAlphaArgs(1)));
ShowContinueError(state, "..requests for IP units from this meter will be ignored.");
}

meter->decMeterNum = decMeterNum;

// This meter is good
int meterNum = op->meters.size();
Expand Down Expand Up @@ -4216,6 +4218,7 @@ void SetInitialMeterReportingAndOutputNames(EnergyPlusData &state,
if (!period.Rpt) {
period.Rpt = true;
if (MeterFileOnlyIndicator) period.RptFO = true;
else op->freqTrackingVariables[(int)freq] = true;
int indexGroupKey = DetermineIndexGroupKeyFromMeterName(state, meter->Name);
std::string indexGroup = DetermineIndexGroupFromMeterGroup(meter);
WriteMeterDictionaryItem(state,
Expand Down

5 comments on commit 9ce5b6a

@nrel-bot
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Qsort (amirroth) - Win64-Windows-10-VisualStudio-16: OK (2726 of 2756 tests passed, 0 test warnings)

Failures:\n

API Test Summary

  • Passed: 14
  • Failed: 1

integration Test Summary

  • Passed: 760
  • Failed: 29

Build Badge Test Badge

@nrel-bot-2
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Qsort (amirroth) - x86_64-Linux-Ubuntu-22.04-gcc-11.4-IntegrationCoverage-Debug: OK (757 of 790 tests passed, 0 test warnings)

Failures:\n

integration Test Summary

  • Passed: 757
  • Failed: 33

Build Badge Test Badge Coverage Badge

@nrel-bot-3
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Qsort (amirroth) - x86_64-MacOS-10.17-clang-14.0.0: OK (3373 of 3543 tests passed, 620 test warnings)

Messages:\n

  • 747 tests had: AUD diffs.
  • 92 tests had: MDD diffs.
  • 169 tests had: MTD diffs.
  • 34 tests had: RDD diffs.
  • 13 tests had: EDD diffs.
  • 70 tests had: ESO big diffs.
  • 68 tests had: Table big diffs.
  • 97 tests had: MTR big diffs.
  • 5 tests had: MTR small diffs.
  • 12 tests had: EIO diffs.
  • 13 tests had: ERR diffs.
  • 1 test had: SSZ small diffs.
  • 1 test had: ZSZ small diffs.
  • 2 tests had: BND diffs.

Failures:\n

API Test Summary

  • Passed: 14
  • Failed: 1

integration Test Summary

  • Passed: 760
  • Failed: 29

regression Test Summary

  • Passed: 647
  • Failed: 139

SQLiteFixture Test Summary

  • Passed: 99
  • Failed: 1

Build Badge Test Badge

@nrel-bot-2b
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Qsort (amirroth) - x86_64-Linux-Ubuntu-22.04-gcc-11.4-UnitTestsCoverage-Debug: OK (1967 of 1969 tests passed, 0 test warnings)

Failures:\n

API Test Summary

  • Passed: 14
  • Failed: 1

SQLiteFixture Test Summary

  • Passed: 99
  • Subprocess aborted: 1

Build Badge Test Badge Coverage Badge

@nrel-bot-2c
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Qsort (amirroth) - x86_64-Linux-Ubuntu-22.04-gcc-11.4: OK (3417 of 3584 tests passed, 624 test warnings)

Messages:\n

  • 751 tests had: AUD diffs.
  • 94 tests had: MDD diffs.
  • 169 tests had: MTD diffs.
  • 35 tests had: RDD diffs.
  • 13 tests had: EDD diffs.
  • 70 tests had: ESO big diffs.
  • 68 tests had: Table big diffs.
  • 98 tests had: MTR big diffs.
  • 5 tests had: MTR small diffs.
  • 12 tests had: EIO diffs.
  • 13 tests had: ERR diffs.
  • 1 test had: SSZ small diffs.
  • 1 test had: ZSZ small diffs.

Failures:\n

API Test Summary

  • Passed: 14
  • Failed: 1

integration Test Summary

  • Passed: 765
  • Failed: 27

regression Test Summary

  • Passed: 667
  • Failed: 139

Build Badge Test Badge

Please sign in to comment.