Skip to content

Conversation

@ayushman1210
Copy link
Contributor

@ayushman1210 ayushman1210 commented Dec 10, 2025

Summary

This update fixes the conflation of soil and microbial maintenance respiration by splitting maintRespiration into:

  • soilMaintRespiration
  • microbeMaintRespiration
    It also corrects the respiration temperature formulation by separating the base rate from the Q10 temperature effect.

Key Changes

  • Replaced maintRespiration with explicit soil and microbe maintenance fluxes.
  • Fixed incorrect mixing of base respiration rate with temperature effect.
  • Updated rSoil calculation:
    • Microbes OFFrSoil = soilMaintRespiration
    • Microbes ONrSoil = microbeMaintRespiration + (1 - microbeEff) * microbeIngestion
  • Updated state.h to match new flux definitions.

Testing

  • Docker build succeeds [success]
  • Model runs with --microbes [success]
  • Output (sipnet.out) generated successfully [success]

Fixes #144

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.

Thanks for putting this together, @ayushman1210! It's almost there, I have a few additions that should be made.

@ayushman1210
Copy link
Contributor Author

hey @Alomir i have made as per your suggestion
please review it
thanks !!

@ayushman1210 ayushman1210 requested a review from Alomir December 13, 2025 09:04
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.

Split maintResp into soilMaintResp and microbeMaintResp for clarity

2 participants