OpenStudioApplication icon indicating copy to clipboard operation
OpenStudioApplication copied to clipboard

Space Infiltration Design Flow Rate instances from Space Type don't show up on Spaces/AIrFlow sub-tab

Open DavidGoldwasser opened this issue 7 years ago • 6 comments

Given a model with ventilation and infiltration assigned at the space type level.

On the Space Types Tab

  • Under "General" view you can see both types of loads.
  • Infiltration can be seen and edited, and deleted from the "Loads" view, but you can't add new ones from this view.

On the Space type

  • Under "Properties/Airflow" you can see inherited ventilation but not inherited infiltration
  • Under "Loads" you can see inherited infiltration but not inherited ventilation.

screen shot 2017-10-02 at 2 22 37 pm

screen shot 2017-10-02 at 2 22 54 pm

screen shot 2017-10-02 at 2 24 29 pm

screen shot 2017-10-02 at 2 25 01 pm

DavidGoldwasser avatar Oct 02 '17 20:10 DavidGoldwasser

Thanks @DavidGoldwasser for reviewing this more closely.

evanweaver avatar Jan 23 '18 19:01 evanweaver

Below are a number of possible changes.

A) I think a minimal solution is to have inherited space infiltration objects show up in the "Properties/Airflow" section of the spaces tab. This will do two things.

  1. It will be more consistent with where what you see for same objects in the space type tab
  2. It will allow a modeler to see ventilation and infiltration loads in a single view (Space/Properties/Airflow) without having to go to (Space/Loads)

B) Since ventilation and infiltration were pulled out of "General" and put in "Airflow" for the spaces tab, it makes sense to add an "Airflow" tab in the Space Types tab and put it between "General" and "Loads".

C) There is also a good argument that "Airflow" on the spaces tab should be pulled out of "Properties" and be put as a new sub-tab between "Properties" and "Loads". A second part of this would be to have the name "General" in space types and "Properties" in spaces unified to same term.

The mockups below reflect changes described in items B and C "Space Types" and "Spaces" would start with "Properties,Airflow, and Loads" and they content they contain would be consistent.

screen shot 2018-01-23 at 2 34 04 pm

screen shot 2018-01-23 at 2 33 52 pm

D) Under loads for both the Space Type tab and Space tab it is strange that infiltration objects show, but not Design Specification OA objects. I'd be in support of an all or none approach (don't show infiltration, or also show DSOA). If there was just a way to add new DSOA and infiltration objects to the loads view similar to other internal loads, then there wouldn't be a need for the "Airflow" section, everything would be viewed and changed in "Loads".

@asparke2 @ljbrackney @macumber and @jmarrec what do you think about this? Is just fixing "A" which is a bug enough, or do you think some minor re-structuring would be beneficial?

DavidGoldwasser avatar Jan 23 '18 21:01 DavidGoldwasser

I agree with A: under the Space tab, the Airflow one should have inherited SpaceType's infiltration design flow rate, DSOA, and space infiltration effective leakage areas.

I understand the rationale of B., and it is confusing right now to put able to put a SpaceInfiltrationDesignFlowRate on the "General" tab, be able to see the object in the IG, schedule is blank, but I have to go to the Loads tab to assign a schedule... So perhaps moving it to a specific tab could give a chance to address this weird behavior.

spacetype_general

spacetype_loads

Point C: I don't really care if it gets moved to a specific tab or a subtab of General, but perhaps you're right, it'd be more consistent. (Side note: the "My Model" in the main right column controller doesn't include "Space Infiltration Design Flow Rate", I have to drag from library... true for SpaceType and Space tabs)

Point D: There's something strange regarding the ZoneVentilationDesignFlowRate. The problem is that this object is placed on ThermalZone, and one TZ can serve multiple spaces... so I'm not sure how we could show that on the space airflow tab...

jmarrec avatar Jan 26 '18 09:01 jmarrec

@jmarrec I updated my note for D. When I said ventilation I mean Design Specification OA objects.

I agree ZoneVentilationDesignFlowRate shouldn't be on the space tab.

DavidGoldwasser avatar Jan 26 '18 21:01 DavidGoldwasser

Related in 1.1.0 infiltration schedules can be deleted from Space Types > Loads using the ✖️ but do not show as deleted in the app.

image

image

MatthewSteen avatar May 14 '21 18:05 MatthewSteen

The last item reported by @MatthewSteen will be fixed in 1.2.0. Rest of this issue is still valid

macumber avatar May 15 '21 02:05 macumber