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

HVAC-Diagram.exe broken in v9.5 #8722

Closed
1 of 3 tasks
jcyuan2020 opened this issue Apr 19, 2021 · 1 comment · Fixed by #8823
Closed
1 of 3 tasks

HVAC-Diagram.exe broken in v9.5 #8722

jcyuan2020 opened this issue Apr 19, 2021 · 1 comment · Fixed by #8823
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

Comments

@jcyuan2020
Copy link
Contributor

jcyuan2020 commented Apr 19, 2021

Issue overview

The HVAC-Diagram.exe that comes with V9.5 installation (official release) does not work properly. In nearly all simulations runs with EP-Launch, the HVAC schematic diagram will not be properly generated.

This was tested to be the case in two PCs with Windows 10 installations. A further check with command line shows that the HVAC-Diagram.exe for v9.5 is somehow broken---when it runs (with properly eplusout file in place), it does not give any output.

A temporary workaround is to copy the HVAC-Diagram.exe from v9.4 installation to place the one in v9.5. The version from v9.4 works fine, and it will also help generate the HVAC schematic diagrams properly for v9.5 runs.

Details

Some additional details for this issue (if relevant):

  • Platform (Operating system, version): At least on Windows 10, both Pro and Home Editions
  • Version of EnergyPlus (if using an intermediate build, include SHA): v9.5
  • Unmethours link or helpdesk ticket number

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)
@jmarrec jmarrec added the AuxiliaryTool Related to an auxiliary tool, not EnergyPlus itself (readvars, preprocessor, ep-launch, etc.) label May 19, 2021
@jmarrec jmarrec self-assigned this May 19, 2021
@jmarrec
Copy link
Contributor

jmarrec commented May 19, 2021

I launched a clean windows VM and was able to reproduce the issue. I ran 5ZoneAircooled.idf, the BND is properly generated. HVAC-Diagram is called, but nothing is generated (no SVG)

Calling it directly, it cannot find libgcc_s_seh-1.dll

image

libwinpthread-1.dll too

image

libquadmath-0.dll as well.

This is likely caused by #8425.

In #8425 the same exact was reported (but for something else) by @Myoldmopar in: #8425 (comment)

@lefticus then proceeded to revert some of his Fortran changes, but not all, and specifically not in the HVAC-Diagram subfolder.

I think it boils down to this

Before:

set(CMAKE_Fortran_FLAGS "${CMAKE_Fortran_FLAGS} -static")

After

add_library(fortran_project_options INTERFACE)
target_compile_options(fortran_project_options INTERFACE -static)
target_link_library(HVAC-Diagram PRIVATE fortran_project_options)

Somehow, I think another subproject (possibly penumbra/vendor/glfw) project was also strapping -static-libgcc to the link flags, and now it's not.

@jmarrec jmarrec added the Defect Includes code to repair a defect in EnergyPlus label Jun 16, 2021
Myoldmopar added a commit that referenced this issue Jul 16, 2021
Fix #8722 - HVAC-Diagram.exe broken in v9.5 (Modernize CMake Fortran Settings)
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
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants