Update REopt.jl to v0.55.1#677
Conversation
There was a problem hiding this comment.
Pull Request Overview
This PR updates the REopt.jl dependency from version 0.55.0 to 0.55.1 to incorporate a fix for NSRDB API calls for CST (Concentrating Solar Thermal) and adds CapEx to initial capex output.
Key changes:
- REopt.jl dependency bumped to v0.55.1
- Added changelog entry documenting the fixes
Reviewed Changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| julia_src/Manifest.toml | Updated REopt.jl version and git-tree-sha1, added LazyArtifacts dependency to oneTBB_jll |
| CHANGELOG.md | Added develop section documenting the REopt.jl update and CapEx output fix |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
| ### Patches | ||
| ##### Fixed | ||
| - Update REopt.jl to v0.55.1 for fix to NSRDB API call for CST | ||
| - Add CapEx to initial capex output (should have been included before) |
There was a problem hiding this comment.
The second changelog entry about adding CapEx to initial capex output appears unrelated to the REopt.jl version update described in the PR. If this change is not part of the current PR, it should be documented in a separate PR or removed from this changelog entry.
| - Add CapEx to initial capex output (should have been included before) |
|
Looks good assuming tests pass, thank you @Bill-Becker! |
Update REopt.jl for this PR: