Skip to content

Conversation

@jmarrec
Copy link
Contributor

@jmarrec jmarrec commented May 26, 2025

Pull request overview

Link to the Ubuntu 22.04 .deb installer to use for CI Testing. If not set, it will default to latest official release.
[OpenStudio Installer]: http://openstudio-ci-builds.s3-website-us-west-2.amazonaws.com/PR-5384/OpenStudio-3.10.0-alpha%2Bf1f36e141c-Ubuntu-22.04-x86_64.deb

This Pull Request is concerning:

  • Case 3 - NewTestForExisting: a new test for an already-existing model API class

Case 3: New test for an already-existing model API class

This adds a test for #5384

Work Checklist

The following has been checked to ensure compliance with the guidelines:

  • Test has been run backwards (see Instructions for Running Docker) for all OpenStudio versions

  • A Matching OSM test has been added with the output of the ruby test for the oldest OpenStudio release where it passes (include OpenStudio Version)

  • Ruby test is stable in the last OpenStudio version: when run multiple times on the same machine, it produces the same total site kBTU.

    • I ensured that I assign systems/loads/etc in a repeatable manner (eg: if I assign stuff to thermalZones, I do model.getThermalZones.sort_by{|z| z.name.to_s}.each do ... so I am sure I put the same ZoneHVAC systems to the same zones regardless of their order)
    • I tested stability using process_results.py (see python process_results.py --help for usage).
      Please paste the text output or heatmap png generated after running the following commands:
      # Clean up all custom-tagged OSWs
      python process_results.py test-stability clean
      # Run your test 5 times in a row. Replace `testname_rb` (eg `airterminal_fourpipebeam_rb`)
      python process_results.py test-stability run -n testname_rb
      # Check that they all passed
      python process_results.py test-status --tagged
      # Check site kBTU differences
      python process_results.py heatmap --tagged
      
  • Object has been added to autosize_hvac.rb to ensure the autosizedXXX values methods do work


Case 4: Other

Please be as explicit as possible about the changes you have made, and why they are warranted.


Review Checklist

  • Code style (indentation, variable names, strip trailing spaces)
  • Functional code review (it has to work!)
  • Matching OSM test has been added or # TODO added to model_tests.rb
  • Appropriate out.osw have been committed
  • Test is stable
  • Object is tested in autosize_hvac as appropriate
  • The appropriate labels have been added to this PR:
    • One of: NewTest, TestFix, NewTestForExisting, Other
    • If NewTest: add PendingOSM or AddedOSM

@jmarrec jmarrec added the NewTestForExisting PR type: a new test for an already-existing model API class label May 26, 2025
@jmarrec jmarrec self-assigned this May 26, 2025
@jmarrec jmarrec force-pushed the 4219-DSOA branch 2 times, most recently from a59c5a5 to 00f46a4 Compare May 26, 2025 15:34
@jmarrec
Copy link
Contributor Author

jmarrec commented May 26, 2025

Ok, running with and without spaces, for 3.7.0 till 3.10.0 alpha

image

jmarrec added 5 commits May 26, 2025 21:00
…y has no DSOA assigned!

The order in which it picks the DSOA (or not) is dependent on the order of spaces... And I have one space without A DSOA at all.
USE_EPLUS_SPACES=false CUSTOMTAG=WithoutSpaces /usr/local/openstudio-3.9.0/bin/openstudio model_tests.rb -n test_space_level_dsoa_py
@jmarrec
Copy link
Contributor Author

jmarrec commented May 26, 2025

I ran as far back as 3.0.0. I modified the test to be more dramatic: each space is EQUALLY sized in floor area. Since it picks up a random space's DSOA in the broken versions (USE_EPLUS_SPACE = true, so 3.5.0 to 3.9.0 included), sometimes it would pick the core one that was like 48 m3/s total airflow, versus 2.0 for the rest of the spaces, making the problem less obvious.

image

@jmarrec
Copy link
Contributor Author

jmarrec commented May 26, 2025

Viewed in %

image

Not sure why the DSOA:SpaceList option has a tiny % difference, but it's much much better!

@jmarrec jmarrec requested a review from joseph-robertson May 26, 2025 21:39
@jmarrec
Copy link
Contributor Author

jmarrec commented May 26, 2025

Test file: 5 equally sized floors, all under the same thermal zone

image

All spaces have a space-assigned DSOA, except the last (Story 5)

image

Zone is served via sys7 (VAV) system:

image

ATU has Control For Outdoor Air = True

image

@jmarrec jmarrec merged commit 590a5d4 into develop May 26, 2025
2 of 3 checks passed
@jmarrec jmarrec deleted the 4219-DSOA branch May 26, 2025 22:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

NewTestForExisting PR type: a new test for an already-existing model API class

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants