Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Transition to v24.1 crashes in the HX transition if there are blank effectiveness fields #10624

Closed
3 tasks done
mjwitte opened this issue Jul 26, 2024 · 0 comments · Fixed by #10639
Closed
3 tasks done
Assignees
Labels
AuxiliaryTool Related to an auxiliary tool, not EnergyPlus itself (readvars, preprocessor, ep-launch, etc.) Defect Includes code to repair a defect in EnergyPlus LowComplexityApproved Used for subcontractor defect complexity requests

Comments

@mjwitte
Copy link
Contributor

mjwitte commented Jul 26, 2024

Issue overview

Got a crash transitioning a user file from 23.2 to 24.1.
image

It's failing on this line in the source:
READ(HxEffectAt75Airflow(i), *) effect75

The incoming HX objects look like this:

HeatExchanger:AirToAir:SensibleAndLatent,
    VMC_1_HEX,               !- Name
    Heat_Recovery_Availability,  !- Availability Schedule Name
    0.305,                   !- Nominal Supply Air Flow Rate {m3/s}
    0.75,                    !- Sensible Effectiveness at 100% Heating Air Flow {dimensionless}
    ,                        !- Latent Effectiveness at 100% Heating Air Flow {dimensionless}
    0.75,                    !- Sensible Effectiveness at 75% Heating Air Flow {dimensionless}
    ,                        !- Latent Effectiveness at 75% Heating Air Flow {dimensionless}
    0.75,                    !- Sensible Effectiveness at 100% Cooling Air Flow {dimensionless}
    ,                        !- Latent Effectiveness at 100% Cooling Air Flow {dimensionless}
    0.75,                    !- Sensible Effectiveness at 75% Cooling Air Flow {dimensionless}
    ,                        !- Latent Effectiveness at 75% Cooling Air Flow {dimensionless}

Workaround

Fill in the default value of 0.0 in the blank HX effectiveness fields in the incoming v23.2 file.

Details

Some additional details for this issue (if relevant):

  • Platform Win11
  • Version of EnergyPlus 24.1
  • Helpdesk ticket number 16730

Checklist

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

  • Defect file added in EnergyPlusDevSupport\DefectFiles
  • Ticket added to EnergyPlus Defect Complexity (Github Project)
  • Pull request created (the pull request will have additional tasks related to reviewing changes that fix this defect)
@jmarrec jmarrec added AuxiliaryTool Related to an auxiliary tool, not EnergyPlus itself (readvars, preprocessor, ep-launch, etc.) Defect Includes code to repair a defect in EnergyPlus labels Aug 5, 2024
@jmarrec jmarrec self-assigned this Aug 5, 2024
jmarrec added a commit that referenced this issue Aug 5, 2024
…that when retrieving HX effectivenesses

This will effectively set the values to 0.0 if blank, since that's what the IDD default is for these fields.
Myoldmopar added a commit that referenced this issue Aug 5, 2024
Fix #10624 - Fix Transition from 23.2.0 to 24.2.0 when a HeatExchanger:AirToAir:SensibleAndLatent has blank effectiveness fields
@Myoldmopar Myoldmopar added the LowComplexityApproved Used for subcontractor defect complexity requests label Aug 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
AuxiliaryTool Related to an auxiliary tool, not EnergyPlus itself (readvars, preprocessor, ep-launch, etc.) Defect Includes code to repair a defect in EnergyPlus LowComplexityApproved Used for subcontractor defect complexity requests
Projects
None yet
3 participants