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

Removed unused zooplankton upward swimming #68

Merged
merged 3 commits into from
Jun 11, 2024

Conversation

charliestock
Copy link
Contributor

@charliestock charliestock commented Jun 11, 2024

This pull request removes some preliminary work to implement upward zooplankton swimming to avoid detrainment. This was ultimately not used in any production codes and will be replaced with a more complete representation of zooplankton movement that Jessica is working on with Mathieu Poupon and Laure Resplandy.

This should not change answers. No changes to the diagnostic table are required, but we should probably delete the following lines (which were already commented out):

"generic_cobalt_z","chl2sfcchl", "chl2sfcchl", "ocean_cobalt_tracers_month_z","all","mean","none",2
"generic_cobalt_z","jupswim_n_Smz", "jupswim_n_Smz","ocean_cobalt_tracers_month_z","all","mean","none",2
"generic_cobalt_z","jupswim_n_Mdz", "jupswim_n_Mdz","ocean_cobalt_tracers_month_z","all","mean","none",2
"generic_cobalt_z","jupswim_n_Lgz", "jupswim_n_Lgz","ocean_cobalt_tracers_month_z","all","mean","none",2
"generic_cobalt_z","vmove_Smz", "vmove_Smz", "ocean_cobalt_tracers_month_z","all","mean","none",2
"generic_cobalt_z","vmove_Mdz", "vmove_Mdz", "ocean_cobalt_tracers_month_z","all","mean","none",2
"generic_cobalt_z","vmove_Lgz", "vmove_Lgz", "ocean_cobalt_tracers_month_z","all","mean","none",2

@yichengt900 yichengt900 self-requested a review June 11, 2024 18:27
Copy link
Collaborator

@yichengt900 yichengt900 left a comment

Choose a reason for hiding this comment

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

@charliestock , Code changes look good, and it passed both 1D and NWA12-RT tests. Approve. I will cleanup the diag_table later.

@yichengt900
Copy link
Collaborator

@charliestock , Code changes look good, and it passed both 1D and NWA12-RT tests. Approve.

@yichengt900 yichengt900 merged commit 7702d91 into dev/cefi Jun 11, 2024
1 check passed
@yichengt900 yichengt900 deleted the remove_zooplankton_swim2/ocean_BGC branch August 1, 2024 15:10
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