Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Questions around pathogen mortality calcs #60

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

MichaelBarryBMT
Copy link
Contributor

Hi guys
These are the queries I had around pathogen mortality please. In summary:

  • The mortality calculation doesn't seem to be consistent with Hipsey et all (2008) equation 9
  • Salinity driven mortality is hardwired to be off
  • Is the /35 included in the user specified salinity factor (see Hipsey et all (2008) eq 9), or does it need to be in the code
    Ta
    Michael

@matthipsey
Copy link
Contributor

Hi MB
I think the salinity effect is included - its computed, and then incremented below to also include temp.
You are correct the coefficient used needs to have the /35 built into it, relative to Eq9 of the original paper.

@matthipsey
Copy link
Contributor

Actually - My apologies, I see that fAOC is 1, so you are right that this will disable it.

So the correction is that fAOC needs to default to 0 .... so that if that term is absent the salinity scaling will occur.

@MichaelBarryBMT
Copy link
Contributor Author

Thanks Matt - This looks good now, except the /35 query on c_SM. Should this be included in the algorithm? Thanks!!

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.

2 participants