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

Space sizing files not saved by EP-Launch #10923

Open
mjwitte opened this issue Feb 6, 2025 · 2 comments
Open

Space sizing files not saved by EP-Launch #10923

mjwitte opened this issue Feb 6, 2025 · 2 comments
Assignees
@JasonGlazer
Labels
AuxiliaryTool Related to an auxiliary tool, not EnergyPlus itself (readvars, preprocessor, ep-launch, etc.) Defect Includes code to repair a defect in EnergyPlus

Comments

@mjwitte
Copy link
Contributor

mjwitte commented Feb 6, 2025 •
edited
Loading

Issue overview

  1. Space sizing output (eplusspsz.csv) was added in Space Sizing and HVAC Part 4 #10566 and support for eplusspsz.csv was added to RunEPlus.bat but not Epl-run.bat. When running a simulation with EP-Launch, this results in the eplusspsz.csv output being stranded in the EPTEMP-* folder and not saved with the other output files. It also prevents the EPTEMP-* folder from being deleted after a simulation.

  2. EP-Launch needs updating to add this file to the View Results buttons.

Operating System (Multiple choices)

Windows

Operating System Version

Any

Version of EnergyPlus

25.1.0-a402772

Unmethours link or helpdesk ticket number

N/A

Defect file

Example file 5ZoneAirCooledWithSpacesHVAC.idf
@mjwitte mjwitte added Triage Issue needs to be assessed and labeled, further information on reported might be needed UnconfirmedDefect Defect has not yet confirmed to be an actual issue Defect Includes code to repair a defect in EnergyPlus and removed UnconfirmedDefect Defect has not yet confirmed to be an actual issue Triage Issue needs to be assessed and labeled, further information on reported might be needed labels Feb 6, 2025
jmarrec added a commit that referenced this issue Feb 10, 2025
@jmarrec
#10923 - Quick fix Epl-run.bat
87561bf
@jmarrec jmarrec mentioned this issue Feb 10, 2025
Merged
20 tasks
@jmarrec
Copy link
Contributor

jmarrec commented Feb 10, 2025 •
edited
Loading

I guess point 2 is for @JasonGlazer ?

Myoldmopar added a commit that referenced this issue Feb 13, 2025
@Myoldmopar
Merge pull request #10932 from NREL/10923
502df3d 
#10923 - Quick fix Epl-run.bat
@JasonGlazer JasonGlazer self-assigned this Feb 17, 2025
@JasonGlazer JasonGlazer added the AuxiliaryTool Related to an auxiliary tool, not EnergyPlus itself (readvars, preprocessor, ep-launch, etc.) label Feb 17, 2025
@JasonGlazer
Copy link
Contributor

Point 2 is definitely a GUI issue and should be pointed to me.

@JasonGlazer JasonGlazer mentioned this issue Feb 17, 2025
Open
jmarrec added a commit that referenced this issue Feb 24, 2025
@jmarrec
#10923 - Quick fix Epl-run.bat
72de825 
Missed some in #10932
cf #10932 (comment)
jmarrec added a commit that referenced this issue Feb 24, 2025
@jmarrec
#10923 - Quick fix Epl-run.bat [skip ci]
229c437 
Missed some in #10932
cf #10932 (comment)
Myoldmopar added a commit that referenced this issue Feb 26, 2025
@Myoldmopar
Merge pull request #10951 from NREL/10923
347c1c0 
#10923 - Quick fix Epl-run.bat
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@JasonGlazer JasonGlazer

Labels
AuxiliaryTool Related to an auxiliary tool, not EnergyPlus itself (readvars, preprocessor, ep-launch, etc.) Defect Includes code to repair a defect in EnergyPlus
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@JasonGlazer @mjwitte @jmarrec