Skip to content

Fixes spaceWidth - #9

Merged
brgix merged 1 commit into
developfrom
width
Feb 3, 2026
Merged

Fixes spaceWidth#9
brgix merged 1 commit into
developfrom
width

Conversation

@brgix

@brgixbrgix commented Feb 3, 2026

Copy link
Copy Markdown
Member

No description provided.

@brgixbrgix self-assigned this Feb 3, 2026
@brgixbrgix added the bug Something isn't working label Feb 3, 2026
Comment threadsrc/osut/osut.py
polyg.reverse()

res = realignedFace(polyg)
res = realignedFace(polyg, True)

Copy link
Copy Markdown
MemberAuthor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A gross omission, inserted more than 12 months ago. Forcing realignFace to rotate so a polygon bounded box is wider than deeper (vs X-axis).

Comment threadtests/test_osut.py
space = openstudio.model.Space(model)
floor = openstudio.model.Surface(vtx, model)
floor.setSpace(space)
self.assertAlmostEqual(osut.spaceWidth(space),1,3)

Copy link
Copy Markdown
MemberAuthor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Basic tests demonstrating that spaceWidth (now) always returns the narrower dimension of a space's bounded box.

@brgix
brgix merged commit 2223fba into developFeb 3, 2026
3 checks passed
@brgix
brgix deleted the width branch February 3, 2026 20:03
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bugSomething isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

@brgix