Skip to content

Commit

Permalink
Merge pull request #10219 from chen1682-ntu-edu-sg/patch-1
Browse files Browse the repository at this point in the history 
api/datatransfer.cc: remove extraneous semicolons (`;`) in `listAllAPIDataCSV`'s output
  • Loading branch information
@Myoldmopar
Myoldmopar authored Sep 21, 2023
2 parents e7398af + 7274e96 commit bf59638
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/EnergyPlus/api/datatransfer.cc
View file
Original file line number Diff line number Diff line change
Expand Up @@ -147,7 +147,7 @@ char *listAllAPIDataCSV(EnergyPlusState state)
output.append("Actuator,");
output.append(availActuator.ComponentTypeName).append(",");
output.append(availActuator.ControlTypeName).append(",");
output.append(availActuator.UniqueIDName).append(";\n");
output.append(availActuator.UniqueIDName).append("\n");
}
output.append("**INTERNAL_VARIABLES**\n");
for (auto const &availVariable : thisState->dataRuntimeLang->EMSInternalVarsAvailable) {
Expand Down

10 comments on commit bf59638

@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.

develop (Myoldmopar) - x86_64-MacOS-10.17-clang-14.0.0: OK (2748 of 2748 tests passed, 0 test warnings)

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.

develop (Myoldmopar) - x86_64-Linux-Ubuntu-22.04-gcc-11.4: OK (2769 of 2769 tests passed, 0 test warnings)

Build Badge Test Badge

@nrel-bot
Copy link

Choose a reason for hiding this comment

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

develop (Myoldmopar) - Win64-Windows-10-VisualStudio-16: OK (2747 of 2747 tests passed, 0 test warnings)

Build Badge Test 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.

develop (Myoldmopar) - x86_64-Linux-Ubuntu-22.04-gcc-11.4-IntegrationCoverage-Debug: OK (789 of 789 tests passed, 0 test warnings)

Build Badge Test Badge Coverage 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.

develop (Myoldmopar) - x86_64-Linux-Ubuntu-22.04-gcc-11.4-UnitTestsCoverage-Debug: OK (1961 of 1961 tests passed, 0 test warnings)

Build Badge Test Badge Coverage Badge

@nrel-bot
Copy link

Choose a reason for hiding this comment

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

10238-AFN-node-temperature (Myoldmopar) - Win64-Windows-10-VisualStudio-16: OK (2747 of 2747 tests passed, 0 test warnings)

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.

10238-AFN-node-temperature (Myoldmopar) - x86_64-Linux-Ubuntu-22.04-gcc-11.4: OK (3574 of 3574 tests passed, 0 test warnings)

Build Badge Test 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.

10238-AFN-node-temperature (Myoldmopar) - x86_64-MacOS-10.17-clang-14.0.0: OK (3533 of 3533 tests passed, 0 test warnings)

Build Badge Test 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.

10238-AFN-node-temperature (Myoldmopar) - x86_64-Linux-Ubuntu-22.04-gcc-11.4-UnitTestsCoverage-Debug: OK (1961 of 1961 tests passed, 0 test warnings)

Build Badge Test Badge Coverage 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.

10238-AFN-node-temperature (Myoldmopar) - x86_64-Linux-Ubuntu-22.04-gcc-11.4-IntegrationCoverage-Debug: OK (789 of 789 tests passed, 0 test warnings)

Build Badge Test Badge Coverage Badge

Please sign in to comment.