Update CMakeLists.txt for 3.11 testing #5513

DavidGoldwasser · 2025-10-28T18:50:01Z

prep for OS 3.11 testing with updated extension and standards gem. There will be at least one more update to standards gem before final OS 3.11.0

Pull request overview

Pull Request Author

Model API Changes / Additions
Any new or modified fields have been implemented in the EnergyPlus ForwardTranslator (and ReverseTranslator as appropriate)
Model API methods are tested (in src/model/test)
EnergyPlus ForwardTranslator Tests (in src/energyplus/Test)
If a new object or method, added a test in NREL/OpenStudio-resources: Add Link
If needed, added VersionTranslation rules for the objects (src/osversion/VersionTranslator.cpp)
Verified that C# bindings built fine on Windows, partial classes used as needed, etc.
All new and existing tests passes
If methods have been deprecated, update rest of code to use the new methods

Labels:

If change to an IDD file, add the label IDDChange
If breaking existing API, add the label APIChange
If deemed ready, add label Pull Request - Ready for CI so that CI builds your PR

Review Checklist

This will not be exhaustively relevant to every PR.

Perform a Code Review on GitHub
Code Style, strip trailing whitespace, etc.
All related changes have been implemented: model changes, model tests, FT changes, FT tests, VersionTranslation, OS App
Labeling is ok
If defect, verify by running develop branch and reproducing defect, then running PR and reproducing fix
If feature, test running new feature, try creative ways to break it
CI status: all green or justified

jmarrec · 2025-11-14T00:09:09Z

@DavidGoldwasser Is there a reason not to merge this? Are you waiting on a standards gem update?

jmarrec · 2025-11-14T16:44:39Z

Ok there are problems.

EmbeddedRuby_Test#test_embedded_gems:

SyntaxError: :/ruby/3.2.0/gems/openstudio-standards-0.8.4/lib/openstudio-standards/btap/costing/daylighting_sensor_control_costing.rb:8: invalid character property name {L}: /(?<=\p{L})(?=\d)/
    eval:255:in `eval'
    eval:255:in `require_embedded_absolute'
    eval:188:in `require'
    <internal::/ruby/3.2.0/rubygems/core_ext/kernel_require.rb>:85:in `require'
    eval:272:in `require_relative'
    :/ruby/3.2.0/gems/openstudio-standards-0.8.4/lib/openstudio-standards.rb:116:in `<module:OpenstudioStandards>'
    :/ruby/3.2.0/gems/openstudio-standards-0.8.4/lib/openstudio-standards.rb:5:in `<main>'
    eval:255:in `eval'
    eval:255:in `require_embedded_absolute'
    eval:240:in `block in require_embedded'
    eval:234:in `each'
    eval:234:in `require_embedded'
    eval:209:in `rescue in require'
    eval:203:in `require'
    <internal::/ruby/3.2.0/rubygems/core_ext/kernel_require.rb>:85:in `require'

https://github.com/NREL/openstudio-standards/blob/5a81f4f55493dc92e3506ae923539c74659d4a67/lib/openstudio-standards/btap/costing/daylighting_sensor_control_costing.rb#L8

not the first time we have an issue with the Oniguruma regex engine in embedded ruby. This will require an openstudio-standards change.

jmarrec · 2025-11-14T17:04:28Z

I'm confused cause this seems to work on my windows VM:

test.rb

standards_template = 'NECB2011'

standards_template = standards_template.gsub(/(?<=\p{L})(?=\d)/, ' ')
puts standards_template

openstudio test.rb

I'm seeing that CI on Ubuntu 24.04 fails to run test_embedded_ruby-embedded_gems, but it works fine on my machine as well..

jmarrec · 2025-11-14T17:09:59Z

I'm hoping that this will suffice:

Fix #2044 - Remove Oniguruma Lookbehind with unicode matcher openstudio-standards#2045

prep for OS 3.11 testing with updated extension and standards gem. There will be at least one more update to standards gem before final OS 3.11.0

Still need to update other OS's with updated hash.

…nstudio-gems/releases/tag/v3.11.0-alfa-2

* NREL/openstudio-gems#121 * NREL/openstudio-standards#2045

jmarrec · 2025-11-17T10:11:08Z

3a86687 : Try with openstudio-standards patched to remove regex unicode lookbehind, which seems to work:

Try openstudio-standards with the regex unicode lookbehind removed openstudio-gems#121
Fix #2044 - Remove Oniguruma Lookbehind with unicode matcher openstudio-standards#2045

…ssage in test

…hance MeasureManagerServer tests for improved error handling and state verification

github-actions · 2025-11-19T22:02:51Z

🧪 Test Results Dashboard

Summary

Metric	Value
Total Tests	4148
Passed	4137
Failed	11
Errors	0
Skipped	0
Success Rate	99.7%
Generated	2025-11-20 00:37:04 UTC

⚠️ Minor Issues Detected

🔍 Failed Tests (11 failures)

Linux-c++ (11 failures)

BCLFixture.BCLMeasure_CTor_throw_invalid_xml.BCLFixture.BCLMeasure_CTor_throw_invalid_xml (run1)

Error Message:

Failed

Full Details:

No details available

OpenStudioCLI.Classic.test_measure_manager.OpenStudioCLI.Classic.test_measure_manager (run1)

Error Message:

Failed

Full Details:

No details available

CLITest-test_bundle-bundle.CLITest-test_bundle-bundle (run1)

Error Message:

Failed

Full Details:

No details available

CLITest-test_bundle-bundle_git.CLITest-test_bundle-bundle_git (run1)

Error Message:

Failed

Full Details:

No details available

CLITest-test_bundle-bundle_native_embedded.CLITest-test_bundle-bundle_native_embedded (run1)

Error Message:

Failed

Full Details:

No details available

CLITest-test_bundle-bundle.CLITest-test_bundle-bundle (run3)

Error Message:

Failed

Full Details:

No details available

CLITest-test_bundle-bundle_git.CLITest-test_bundle-bundle_git (run3)

Error Message:

Failed

Full Details:

No details available

CLITest-test_bundle-bundle_native_embedded.CLITest-test_bundle-bundle_native_embedded (run3)

Error Message:

Failed

Full Details:

No details available

CLITest-test_bundle-bundle.CLITest-test_bundle-bundle (run2)

Error Message:

Failed

Full Details:

No details available

CLITest-test_bundle-bundle_git.CLITest-test_bundle-bundle_git (run2)

Error Message:

Failed

Full Details:

No details available

CLITest-test_bundle-bundle_native_embedded.CLITest-test_bundle-bundle_native_embedded (run2)

Error Message:

Failed

Full Details:

No details available

📊 Test Run Information

Run	XML File	Status
run1	`results.xml`	✅ Found
run3	`results.xml`	✅ Found
run2	`results.xml`	✅ Found

…st handling - Introduced .bundle/config for Bundler settings. - Updated .gitignore to include .bundle/config. - Added comprehensive test utility functions for CI and local development. - Implemented retry logic for network-dependent bundle installs in tests. - Enhanced CMake configuration to label network-dependent and flaky tests.

…proved stability

ci-commercialbuildings · 2025-11-20T00:56:00Z

CI Results for 4fed640:

Ubuntu 24.04:
Ubuntu 22.04:
Darwin:
- OpenStudio-3.11.0-alpha+3383546343-Darwin-x86_64.dmg
- OpenStudio-3.11.0-alpha+3383546343-Darwin-x86_64.tar.gz
Windows:
- OpenStudio-3.11.0-alpha+9f94cb7d4c-Windows.exe
- OpenStudio-3.11.0-alpha+9f94cb7d4c-Windows.tar.gz
Code Coverage:

tijcolem added the Pull Request - Ready for CI This pull request if finalized and is ready for continuous integration verification prior to merge. label Oct 29, 2025

DavidGoldwasser added this to the OpenStudio SDK 3.11.0 milestone Oct 30, 2025

jmarrec mentioned this pull request Nov 14, 2025

Update gems for os311 testing NREL/openstudio-gems#120

Merged

jmarrec self-requested a review November 14, 2025 16:00

jmarrec force-pushed the v3.11.0-OS-gems-update branch from 9788f58 to 807bf65 Compare November 14, 2025 16:01

jmarrec mentioned this pull request Nov 14, 2025

Remove Oniguruma Lookbehind with unicode matcher and similar constructs NREL/openstudio-standards#2044

Closed

DavidGoldwasser and others added 6 commits November 17, 2025 09:21

Update CMakeLists.txt

f91bf6a

prep for OS 3.11 testing with updated extension and standards gem. There will be at least one more update to standards gem before final OS 3.11.0

updating hash for OS Gems for Linux and Windows

961075e

Still need to update other OS's with updated hash.

updating hash fro Mac Intel

804d602

updating remaining md5 hashes for gems

92e5fc4

Update to alfa2 wich bumps tbd/osut/oslg: https://github.com/NREL/ope…

a28b129

…nstudio-gems/releases/tag/v3.11.0-alfa-2

Try with openstudio-standards patched to remove regex unicode lookbehind

3a86687

* NREL/openstudio-gems#121 * NREL/openstudio-standards#2045

jmarrec force-pushed the v3.11.0-OS-gems-update branch from b7062d6 to 3a86687 Compare November 17, 2025 08:24

jmarrec mentioned this pull request Nov 17, 2025

Fix #2044 - Remove Oniguruma Lookbehind with unicode matcher NREL/openstudio-standards#2045

Merged

16 tasks

DavidGoldwasser assigned anchapin and jmarrec Nov 18, 2025

anchapin added 2 commits November 19, 2025 16:13

Update Gemfiles to use HTTPS for RubyGems source and improve error me…

b2e5f89

…ssage in test

Update Jenkinsfile to use specific version of shared libraries and en…

992818f

…hance MeasureManagerServer tests for improved error handling and state verification

anchapin added 2 commits November 19, 2025 19:23

Update Jenkinsfile to use specific version of shared libraries for im…

4fed640

…proved stability

jmarrec mentioned this pull request Nov 20, 2025

Try openstudio-standards with the regex unicode lookbehind removed NREL/openstudio-gems#121

Merged

jmarrec merged commit 0b0d540 into develop Nov 20, 2025
5 of 6 checks passed

jmarrec deleted the v3.11.0-OS-gems-update branch November 20, 2025 12:08

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Update CMakeLists.txt for 3.11 testing #5513

Update CMakeLists.txt for 3.11 testing #5513

Uh oh!

DavidGoldwasser commented Oct 28, 2025 •

edited by jmarrec

Loading

Uh oh!

jmarrec commented Nov 14, 2025

Uh oh!

jmarrec commented Nov 14, 2025 •

edited

Loading

Uh oh!

jmarrec commented Nov 14, 2025 •

edited

Loading

Uh oh!

jmarrec commented Nov 14, 2025

Uh oh!

jmarrec commented Nov 17, 2025 •

edited

Loading

Uh oh!

github-actions bot commented Nov 19, 2025 •

edited

Loading

Uh oh!

ci-commercialbuildings commented Nov 20, 2025 •

edited

Loading

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

6 participants

Update CMakeLists.txt for 3.11 testing #5513

Update CMakeLists.txt for 3.11 testing #5513

Uh oh!

Conversation

DavidGoldwasser commented Oct 28, 2025 • edited by jmarrec Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Pull request overview

Pull Request Author

Review Checklist

Uh oh!

jmarrec commented Nov 14, 2025

Uh oh!

jmarrec commented Nov 14, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

jmarrec commented Nov 14, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

jmarrec commented Nov 14, 2025

Uh oh!

jmarrec commented Nov 17, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

github-actions bot commented Nov 19, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

🧪 Test Results Dashboard

Summary

⚠️ Minor Issues Detected

🔍 Failed Tests (11 failures)

Linux-c++ (11 failures)

📊 Test Run Information

Uh oh!

ci-commercialbuildings commented Nov 20, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

6 participants

DavidGoldwasser commented Oct 28, 2025 •

edited by jmarrec

Loading

jmarrec commented Nov 14, 2025 •

edited

Loading

jmarrec commented Nov 14, 2025 •

edited

Loading

jmarrec commented Nov 17, 2025 •

edited

Loading

github-actions bot commented Nov 19, 2025 •

edited

Loading

ci-commercialbuildings commented Nov 20, 2025 •

edited

Loading