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.

Sign up for GitHub

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

Jump to bottom

HospitalBaseline.idf sometimes times out due to expensive calls inside BuildKeyVarList #9550

Closed
1 of 2 tasks
jmarrec opened this issue Jul 19, 2022 · 0 comments · Fixed by #9556
Closed
1 of 2 tasks

HospitalBaseline.idf sometimes times out due to expensive calls inside BuildKeyVarList #9550

jmarrec opened this issue Jul 19, 2022 · 0 comments · Fixed by #9556
Assignees
@jmarrec
Labels
Defect Includes code to repair a defect in EnergyPlus

Comments

@jmarrec
Copy link
Contributor

jmarrec commented Jul 19, 2022

Issue overview

HospitalBaseline.idf sometimes times out on CI.

I noticed it appears to spent too much time in BuildKeyVarList

Details

Some additional details for this issue (if relevant):

  • Platform (Operating system, version): All
  • Version of EnergyPlus (if using an intermediate build, include SHA): develop, 0c9b8c4

Checklist

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

  • [c] Defect file added: HospitalBaseline.idf
  • 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)
@jmarrec jmarrec added the Defect Includes code to repair a defect in EnergyPlus label Jul 19, 2022
@jmarrec jmarrec self-assigned this Jul 19, 2022
@jmarrec jmarrec mentioned this issue Jul 22, 2022
Merged
20 tasks
Myoldmopar added a commit that referenced this issue Aug 2, 2022
@Myoldmopar
Merge pull request #9556 from NREL/BuildKeyVar_perf
a962e75 
Fix #9550 - Avoid expensive calls inside BuildKeyVarList
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jmarrec jmarrec

Labels
Defect Includes code to repair a defect in EnergyPlus
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix #9550 - Avoid expensive calls inside BuildKeyVarList NREL/EnergyPlus
1 participant
@jmarrec