Skip to content

Commit

Permalink
Fixes #8372 - Incorrect transition of RoomAir:Node:AirflowNetwork:HVA…
Browse files Browse the repository at this point in the history
  • Loading branch information
jmarrec committed Dec 8, 2020
1 parent 7f35ab6 commit 7df1389
Showing 1 changed file with 10 additions and 9 deletions.
19 changes: 10 additions & 9 deletions src/Transition/CreateNewIDFUsingRulesV9_3_0.f90
Original file line number Diff line number Diff line change
Expand Up @@ -684,17 +684,18 @@ SUBROUTINE CreateNewIDFUsingRules(EndOfFile,DiffOnly,InLfn,AskForInput,InputFile
! Wait - there's more for the transition for AirTerminal:SingleDuct:Uncontrolled
CASE('ROOMAIR:NODE:AIRFLOWNETWORK:HVACEQUIPMENT')
IF(TotATSDUObjs > 0) THEN
NoDiff = .true.
CALL GetNewObjectDefInIDD(ObjectName,NwNumArgs,NwAorN,NwReqFld,NwObjMinFlds,NwFldNames,NwFldDefaults,NwFldUnits)
OutArgs(1) = InArgs(1)
! Loop through fields looking for AirTerminal:SingleDuct:Uncontrolled node name
DO nodeCount=2, CurArgs
IF (SameString(TRIM(InArgs(nodeCount)), TRIM(ATSDUNodeNames(atCount)))) THEN
OutArgs(nodeCount) = TRIM(InArgs(nodeCount)) // ' ATInlet'
ELSE
OutArgs(nodeCount) = InArgs(nodeCount)
OutArgs(1:CurArgs) = InArgs(1:CurArgs)
! Loop through "ZoneHVAC or Air Terminal Equipment Object Type" fields looking for
! "AirTerminal:SingleDuct:Uncontrolled" and replace with "AirTerminal:SingleDuct:ConstantVolume:NoReheat"
! Four extensible fields
DO CurField=2, CurArgs-1, 4
IF (SameString(TRIM(InArgs(CurField)),'AirTerminal:SingleDuct:Uncontrolled')) THEN
OutArgs(CurField) = 'AirTerminal:SingleDuct:ConstantVolume:NoReheat'
NoDiff = .false.
END IF
ENDDO
NoDiff = .false.
END DO
ELSE
CALL GetNewObjectDefInIDD(ObjectName,NwNumArgs,NwAorN,NwReqFld,NwObjMinFlds,NwFldNames,NwFldDefaults,NwFldUnits)
OutArgs(1:CurArgs)=InArgs(1:CurArgs)
Expand Down

5 comments on commit 7df1389

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

8372_Transition_RoomAirflowNetwork_v920_v930 (jmarrec) - x86_64-MacOS-10.15-clang-11.0.0: OK (2991 of 2992 tests passed, 0 test warnings)

Failures:\n

EnergyPlusFixture Test Summary

  • Passed: 1090
  • Failed: 1

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.

8372_Transition_RoomAirflowNetwork_v920_v930 (jmarrec) - x86_64-Linux-Ubuntu-18.04-gcc-7.5: OK (3031 of 3032 tests passed, 0 test warnings)

Messages:\n

  • 1 test had: EIO diffs.
  • 1 test had: ESO small diffs.
  • 1 test had: MTR small diffs.
  • 1 test had: Table big diffs.

Failures:\n

regression Test Summary

  • Passed: 735
  • 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.

8372_Transition_RoomAirflowNetwork_v920_v930 (jmarrec) - x86_64-Linux-Ubuntu-18.04-gcc-7.5-UnitTestsCoverage-Debug: OK (1558 of 1558 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.

8372_Transition_RoomAirflowNetwork_v920_v930 (jmarrec) - Win64-Windows-10-VisualStudio-16: OK (2249 of 2249 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.

8372_Transition_RoomAirflowNetwork_v920_v930 (jmarrec) - x86_64-Linux-Ubuntu-18.04-gcc-7.5-IntegrationCoverage-Debug: Coverage Too Low

Build Badge Test Badge Coverage Badge

Please sign in to comment.