Skip to content

Non-matching field name and value in error messages #10137

@yujiex

Description

@yujiex

Issue overview

In the VariableSpeedCoils.cc, the following error message aims to show the current value of a field is invalid. The index i and j should be the same.

format("{}{}=\"{}\", invalid", RoutineName, CurrentModuleObject, state.dataVariableSpeedCoils->VarSpeedCoil(DXCoilNum).Name));
ShowContinueError(state, format("...not found {}=\"{}\".", cAlphaFields(i), AlphArray(j)));

A few instances were found where these indices are different, i.e. the name of the field and the value of the field are not matching:

ShowContinueError(state, format("...not found {}=\"{}\".", cAlphaFields(14), AlphArray(10)));

// AlfaFieldIncre is defined as AlfaFieldIncre = 9 + (I - 1) * 7
ShowContinueError(state, format("...not found {}=\"{}\".", cAlphaFields(AlfaFieldIncre), AlphArray(14 + (I - 1) * 6)));

// AlfaFieldIncre is defined as 11 + (I - 1) * 7
ShowContinueError(state, format("...not found {}=\"{}\".", cAlphaFields(AlfaFieldIncre), AlphArray(16 + (I - 1) * 6)));

Checklist

Add to this list or remove from it as applicable. This is a simple templated set of guidelines.

  • Defect file added (list location of defect file here)
  • Ticket added to Pivotal for defect (development team task)
  • Pull request created (the pull request will have additional tasks related to reviewing changes that fix this defect)

Metadata

Metadata

Assignees

Labels

DefectIncludes code to repair a defect in EnergyPlus

Type

No type

Projects

No projects

Relationships

None yet

Development

No branches or pull requests

Issue actions