Skip to content

Conversation

@Bill-Becker
Copy link
Collaborator

Update REopt.jl for this PR:

@Bill-Becker Bill-Becker requested a review from Copilot October 21, 2025 20:47
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

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)
Copy link

Copilot AI Oct 21, 2025

Choose a reason for hiding this comment

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

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.

Suggested change
- Add CapEx to initial capex output (should have been included before)

Copilot uses AI. Check for mistakes.
@Bill-Becker Bill-Becker requested a review from zolanaj October 21, 2025 20:49
@zolanaj
Copy link
Collaborator

zolanaj commented Oct 21, 2025

Looks good assuming tests pass, thank you @Bill-Becker!

@Bill-Becker Bill-Becker merged commit 796e7a9 into develop Oct 21, 2025
1 check passed
@Bill-Becker Bill-Becker deleted the reoptjl-0551 branch October 21, 2025 22:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants