Conversation
…stem-Model/AtlantisTrunk into contaminanttiming
Merge commits in main back to dev so it is current
Contaminant uptake
|
@eafulton This commit includes a bug fix for your review to the contaminant code. In the Puget Sound Model and NoBA there were issues with some invertebrate and macroalgae groups not taking up contaminants. First, we thought it was just a problem with parametrization because the Salish Sea model didn't have that problem. After trying different parameter combinations, we realized the model differences were an issue of how the models were set up to favor uptake (PAHs) vs transfer (mostly PCBs and Mercury). We then identified several issues, including feeding vs. when prey had contaminants, the timing of contaminants being taken up and written into the array, and dead organisms' contaminants not being transferred to carrion. |
|
@eafulton ready for your review, see comment above. Talked to @cecilieha yesterday, and she has tested the bug fix in NoBA and it solved all her issues. Her invert groups now uptake contaminants. |
PRs into dev contained all info we needed. They have already been reviewed. If you've got this far we're in good shape!
List PRs
List the PRs that made it into dev that are waiting to be pulled into main:
Types of changes
What types of changes does code introduce? Put an
xin the boxes that apply.This will inform the new release number.
Reviewer instructions
The underlying PRs should already have been reviewed.
If this is a New Feature, Breaking change, or a bug fix, the Atlantis Manual will need to be updated to reflect these changes.
Now would be a good time to create an issue in the Atlantis Manual repo and begin documenting your changes.