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

CapFTCurveIndex is always set to speed 1 for multi-speed DX coils design size #7625

Closed
3 tasks
Nigusse opened this issue Nov 19, 2019 · 0 comments · Fixed by #7626
Closed
3 tasks

CapFTCurveIndex is always set to speed 1 for multi-speed DX coils design size #7625

Nigusse opened this issue Nov 19, 2019 · 0 comments · Fixed by #7626
Assignees
Labels
Defect Includes code to repair a defect in EnergyPlus

Comments

@Nigusse
Copy link
Contributor

Nigusse commented Nov 19, 2019

Issue overview

The function GetDXCoilCapFTCurveIndex() always sets to the speed 1 (lowest speed) capacity function of temperature curve index for MULTI-SPEED DX coils. The correct curve index should have been the maximum speed number. This is a defect, incorrect curve index may slightly change the design size values if the curves are different for different speed. This function is commonly used when sizing DX coils from parent object.

Details

Some additional details for this issue (if relevant):

  • Platform (all)
  • Version of EnergyPlus (current develop)
  • Observed while working on other issues

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)
@Nigusse Nigusse added the Defect Includes code to repair a defect in EnergyPlus label Nov 19, 2019
@Nigusse Nigusse self-assigned this Nov 21, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Defect Includes code to repair a defect in EnergyPlus
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant