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

Fe #77

Open
eclare108213 opened this issue Jan 1, 2018 · 5 comments
Open

Fe #77

eclare108213 opened this issue Jan 1, 2018 · 5 comments

Comments

@eclare108213
Copy link
Contributor

Dissolved and particulate iron seems to be labeled inconsistently in the code and also in the documentation. In some places it is fed, fep, others dFe, pFe. Can we make all of these references or labels or variable names consistent, or is there a real difference in them?

@njeffery
Copy link
Contributor

njeffery commented Jan 2, 2018

@eclare108213 : The names are correct in the documentation, but yes, they could be changed in the code. fed and fep are the ocean source values. dFe and pFe are used more commonly throughout to refer to ice quantities. It's probably simplest to change fed to dfe and fep to pfe in the code and documentation.

@eclare108213
Copy link
Contributor Author

I didn't realize that ocean and ice location was distinguishing them. Should we be more explicit about that, e.g. dFei and dFeo?

@njeffery
Copy link
Contributor

njeffery commented Jan 2, 2018

The other tracers refer to their ocean values using three lower case letters (eg. nit or sil). So both fed or dfe is consistent if not particularly clear. The more I look at the code, the more I see examples of "Fed_ and Fep_" refering to related iron quantities in the ice.

@eclare108213
Copy link
Contributor Author

@njeffery This is a very old issue in the Icepack repo -- was the naming convention made more consistent in the new BGC code?

@njeffery
Copy link
Contributor

njeffery commented Aug 28, 2024 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants