Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Upgrade to OpenStudio 3.9 and Ruby 3.2.2 #155

Merged
merged 59 commits into from
Jan 27, 2025
Merged

Upgrade to OpenStudio 3.9 and Ruby 3.2.2 #155

merged 59 commits into from
Jan 27, 2025

Conversation

vtnate
Copy link
Collaborator

@vtnate vtnate commented Nov 13, 2024

Resolves #[issue number here]

Pull Request Description

  • Update to OpenStudio 3.9
  • Necessitates an update to Ruby 3.2.2
  • Temporarily use branches of OS dependencies during testing

Checklist (Delete lines that don't apply)

  • Unit tests have been added or updated
  • Documentation has been modified appropriately
  • All ci tests pass (green)
  • An issue has been created (which will be used for the changelog)
  • This branch is up-to-date with develop

@vtnate vtnate added enhancement New feature or request Clean GEMFILE before merge This uses a temporary dependency version breaking-change This update is not backward compatible labels Nov 13, 2024
@vtnate vtnate self-assigned this Nov 13, 2024
@coveralls
Copy link

coveralls commented Nov 13, 2024

Pull Request Test Coverage Report for Build 12997113022

Details

  • 13 of 16 (81.25%) changed or added relevant lines in 5 files are covered.
  • 5 unchanged lines in 2 files lost coverage.
  • Overall coverage decreased (-0.4%) to 75.994%

Changes Missing Coverage Covered Lines Changed/Added Lines %
lib/urbanopt/reopt/feature_report_adapter.rb 2 3 66.67%
lib/urbanopt/reopt/reopt_ghp_post_processor.rb 5 6 83.33%
lib/urbanopt/reopt/scenario_report_adapter.rb 4 5 80.0%
Files with Coverage Reduction New Missed Lines %
lib/urbanopt/reopt/reopt_ghp_post_processor.rb 1 89.29%
lib/urbanopt/reopt/feature_report_adapter.rb 4 72.12%
Totals Coverage Status
Change from base Build 12901847585: -0.4%
Covered Lines: 994
Relevant Lines: 1308

💛 - Coveralls

vtnate and others added 4 commits December 19, 2024 12:41
* use released versions of gems

* bump to v1.0.0

* add year to district input for ElectricLoad

* add year to ElectricLoad for GHP assumptions

* add year to ElectricLoad test assumption files

* use `v3` in ghp test api url instead of `stable`

* rubocop formatting

* remove unused variable

* less action & more assertion in 15-minute-interval test
@vtnate vtnate requested a review from kflemin January 27, 2025 19:43
@kflemin kflemin merged commit 4038888 into develop Jan 27, 2025
1 check passed
@kflemin kflemin deleted the os39 branch January 27, 2025 20:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking-change This update is not backward compatible Clean GEMFILE before merge This uses a temporary dependency version enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants