Skip to content

Add an ENV Variable to force space load instances translation or not - #234

Open
jmarrec wants to merge 1 commit into
developfrom
SpaceLoadInstances
Open

Add an ENV Variable to force space load instances translation or not#234
jmarrec wants to merge 1 commit into
developfrom
SpaceLoadInstances

Conversation

@jmarrec

@jmarrec jmarrec commented Sep 10, 2026

Copy link
Copy Markdown
Contributor

Pull request overview

cf NatLabRockies/OpenStudio#5651

This Pull Request is concerning:

  • Case 4 - Other: Something else, like maintenance of the repo, or just committing test results with a new OpenStudio version.

Depending on your answer, please fill out the required section below, and delete the three others.
Leave the review checklist in place.


Case 4: Other

You can now use USE_SPACE_LOAD_INSTANCES=false to turn off FT Translation to SpaceLoad Instances, eg it won't translate to ElectricEquipment:Instance + ElectricEquipment:Definition but to ElectricEquipment


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 self-assigned this Sep 10, 2026
@jmarrec jmarrec added the Other PR type: Something else, like maintenance of the repo, or just committing test results label Sep 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Other PR type: Something else, like maintenance of the repo, or just committing test results

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant