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

Adding Plenum height causes Model running failure #294

Closed
LouYingli opened this issue Jan 11, 2021 · 2 comments
Closed

Adding Plenum height causes Model running failure #294

LouYingli opened this issue Jan 11, 2021 · 2 comments

Comments

@LouYingli
Copy link

Issue overview

Adding Plenum height causes Model running failure.

Current Behavior

After adding the Plenum height, the simulation fails to run.

Expected Behavior

After adding the Plenum height, the simulation should run well.

Steps to Reproduce

  1. Add Plenum height in the Geometry/Editor/Story.
  2. Click Merge with Current OSM.
  3. Click run.
    Following is the video link of this bug.
    https://drive.google.com/file/d/1ktQqWDNuxEvNIjGbz2CheoUnfoL3SCvL/view?usp=sharing

Details

Environment

  • Platform (Operating system, version): Windows 10 Pro (1909) for Workstations
  • Version of OpenStudioApplication: OpenStudio Application 1.0.0
@LouYingli LouYingli added the Triage Issue needs to be assessed and labeled, further information on reported might be needed label Jan 11, 2021
@macumber macumber added component - Geometry Editor severity - Major Bug 💥 and removed Triage Issue needs to be assessed and labeled, further information on reported might be needed labels Jan 17, 2021
@macumber macumber added this to the OpenStudio Application 1.2.0 milestone Jan 17, 2021
@macumber
Copy link
Collaborator

Thanks for the report, this seems like a bad issue.

@macumber
Copy link
Collaborator

I think there are two separate issues here.

First issue:

  1. Create floor plan, merge with model
  2. Set building construction set
  3. Return to floor plan, merge with model
  4. Building construction set is removed

Second issue:

  1. Create floor plan, add plenum height, merge with model
  2. Assign thermal zone to plenum
  3. Return to floor plan, merge with model
  4. Additional thermal zones are created

macumber added a commit that referenced this issue May 28, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants