Skip to content

SIP128 Add fert event handling (C only) - #131

Merged
dlebauer merged 10 commits into
masterfrom
SIP128_Add_Fert_Event_Carbon_Only
Aug 1, 2025
Merged

SIP128 Add fert event handling (C only)#131
dlebauer merged 10 commits into
masterfrom
SIP128_Add_Fert_Event_Carbon_Only

Conversation

@Alomir

@Alomir Alomir commented Jul 21, 2025

Copy link
Copy Markdown
Collaborator

This PR:

  • Adds basic fertilization event handling (just C, no N)
  • Updates harvest handling for the no-litter-pool case
  • Updates tests for harvest change
  • Updates output formatting for unit tests
  • Adds test for basic fert event

@Alomir
Alomir marked this pull request as ready for review July 21, 2025 22:25
@Alomir Alomir assigned Alomir and unassigned dlebauer and infotroph Jul 22, 2025
@Alomir
Alomir requested review from dlebauer and infotroph July 22, 2025 19:05
@dlebauer

Copy link
Copy Markdown
Member

Added documentation as a PR to this one; plz confirm correctness and completeness #135

@dlebauer dlebauer left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks great! Good to go once documentation in #135 is incorporated.

dlebauer and others added 2 commits August 1, 2025 10:42
…is off (#135)

* updated documentation to describe what happens when LITTER_POOL=0

* remove redundant explanation
@dlebauer
dlebauer merged commit e7017ca into master Aug 1, 2025
9 checks passed
@dlebauer
dlebauer deleted the SIP128_Add_Fert_Event_Carbon_Only branch August 1, 2025 16:46
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.

Fix event implementation when litter pool is off Add fertilization event for carbon (only)

3 participants