Skip to content

Fix typo and incorrect variable name in balance.c comment#254

Merged
Alomir merged 27 commits intomasterfrom
copilot/sub-pr-248-another-one
Feb 6, 2026
Merged

Fix typo and incorrect variable name in balance.c comment#254
Alomir merged 27 commits intomasterfrom
copilot/sub-pr-248-another-one

Conversation

Copy link
Contributor

Copilot AI commented Feb 6, 2026

Final review of PR #248 identified a typo in src/sipnet/balance.c line 21.

Changes

  • Fixed typo: "we ust plantWoodC" → "we use plantWoodC"
  • Corrected variable name reference: "plantNSCWoodCDelta" → "plantWoodCStorageDelta"

The comment now accurately describes the nitrogen accounting logic where plantWoodC is used without the storage delta component.

// Note: this is the one place where we use plantWoodC by itself; it's the
// reason plantWoodCStorageDelta was created, so that we can ignore it here.

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Co-authored-by: dlebauer <464871+dlebauer@users.noreply.github.com>
Copilot AI changed the title [WIP] Add carbon and nitrogen balance checks Fix typo and incorrect variable name in balance.c comment Feb 6, 2026
Copilot AI requested a review from dlebauer February 6, 2026 05:45
Base automatically changed from add_carbon_balance_check to master February 6, 2026 17:02
@Alomir Alomir marked this pull request as ready for review February 6, 2026 18:08
Copy link
Collaborator

@Alomir Alomir left a comment

Choose a reason for hiding this comment

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

LGTM

@Alomir Alomir merged commit 25121f2 into master Feb 6, 2026
11 checks passed
@Alomir Alomir deleted the copilot/sub-pr-248-another-one branch February 6, 2026 18:09
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