Skip to content

Add new inputs for single owner to control application to debt service#1398

Merged
brtietz merged 2 commits into
developfrom
non_energy_revenue_ds_checkboxes
Jun 1, 2026
Merged

Add new inputs for single owner to control application to debt service#1398
brtietz merged 2 commits into
developfrom
non_energy_revenue_ds_checkboxes

Conversation

@brtietz
Copy link
Copy Markdown
Collaborator

@brtietz brtietz commented May 19, 2026

Pull Request Template

Description

Stakeholder feedback on non-energy revenue inputs was to add a debt service checkbox. This PR adds that checkbox.

Corresponding branches and PRs:

non_energy_revenue_ds_checkboxes on SAM (NatLabRockies/SAM#2194) develop on other branches.

Unit Test Impact:

New test for cmod_singleowner to capture this behavior. No expected impacts on other tests.

Checklist

  • requires help revision and I added that label
  • adds, removes, modifies, or deletes variables in existing compute modules
  • adds a new compute module
  • changes defaults
  • I've tagged this PR to a milestone

@coveralls
Copy link
Copy Markdown

coveralls commented May 19, 2026

Coverage Report for CI Build 26655992503

Coverage decreased (-0.002%) to 56.395%

Details

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

Uncovered Changes

No uncovered changes found.

Coverage Regressions

48 previously-covered lines in 2 files lost coverage.

File Lines Losing Coverage Coverage
ssc/ssc/cmod_singleowner.cpp 43 92.04%
ssc/shared/lib_geothermal.cpp 5 65.99%

Coverage Stats

Coverage Status
Relevant Lines: 121685
Covered Lines: 68624
Line Coverage: 56.39%
Coverage Strength: 3440701.49 hits per line

💛 - Coveralls

@brtietz brtietz merged commit 2226dc4 into develop Jun 1, 2026
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants