Skip to content

SDKtool CMakeLists capitalization error#1399

Merged
sjanzou merged 2 commits into
developfrom
agrivoltaics_beta_5_15_26
May 27, 2026
Merged

SDKtool CMakeLists capitalization error#1399
sjanzou merged 2 commits into
developfrom
agrivoltaics_beta_5_15_26

Conversation

@sjanzou
Copy link
Copy Markdown
Collaborator

@sjanzou sjanzou commented May 22, 2026

Fix CMake generation error on MacOS

@sjanzou sjanzou self-assigned this May 22, 2026
@sjanzou sjanzou requested review from brtietz and cpaulgilman May 22, 2026 05:00
@coveralls
Copy link
Copy Markdown

Coverage Report for CI Build 26269238104

Coverage decreased (-0.05%) to 56.281%

Details

  • Coverage decreased (-0.05%) from the base build.
  • Patch coverage: No coverable lines changed in this PR.
  • 4042 coverage regressions across 41 files.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

4042 previously-covered lines in 41 files lost coverage.

Top 10 Files by Coverage Loss Lines Losing Coverage Coverage
ssc/tcs/csp_solver_lf_dsg_collector_receiver.cpp 656 0.0%
ssc/tcs/csp_dispatch.cpp 491 0.46%
ssc/tcs/csp_solver_core.cpp 491 62.94%
ssc/tcs/csp_solver_trough_collector_receiver.cpp 441 73.94%
ssc/ssc/cmod_tcsmolten_salt.cpp 275 72.84%
ssc/tcs/csp_solver_piston_cylinder_tes.cpp 254 1.25%
ssc/tcs/csp_dispatch_ortools.cpp 186 75.75%
ssc/tcs/csp_solver_pc_Rankine_indirect_224.cpp 155 61.15%
ssc/ssc/cmod_trough_physical.cpp 147 75.72%
ssc/ssc/cmod_mspt_iph.cpp 108 0.0%

Coverage Stats

Coverage Status
Relevant Lines: 121316
Covered Lines: 68278
Line Coverage: 56.28%
Coverage Strength: 3450387.58 hits per line

💛 - Coveralls

Copy link
Copy Markdown
Collaborator

@cpaulgilman cpaulgilman left a comment

Choose a reason for hiding this comment

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

Looks good. I searched for other instances of "SDKTool" but didn't find any that aren't in comments.

@sjanzou sjanzou merged commit f0c7437 into develop May 27, 2026
8 checks passed
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.

4 participants