Skip to content

Minor fixes, typos - #30

Merged
brgix merged 2 commits into
developfrom
fixes
Apr 22, 2026
Merged

Minor fixes, typos#30
brgix merged 2 commits into
developfrom
fixes

Conversation

@brgix

@brgixbrgix commented Apr 21, 2026

Copy link
Copy Markdown
Member

Fixes typos and other minor issues, identified while updating pyOSut.

No meaningful change in results.

@brgixbrgix self-assigned this Apr 21, 2026
@brgixbrgix added the bug Something isn't working label Apr 21, 2026
Comment threadlib/osut/utils.rb

unless mt.setThermalResistance(r)
return invalid("Failed #{id}: RSi#{r.round(2)}", mth)
return invalid("Failed #{id}: RSi#{r.round(2)}", mth, 0, DBG, 0.0)

@brgixbrgixApr 21, 2026

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.

Method resetUo() should always return a real number - not the OSlg default nil.


r1 = OpenStudio::Model::PlanarSurface.stillAirFilmResistance(tilt) * 2
r2 = surface.filmResistance
r2 = adjacent.filmResistance

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.

Better test.

@brgix
brgix merged commit 537c0de into developApr 22, 2026
7 checks passed
@brgix
brgix deleted the fixes branch April 22, 2026 18:44
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