Skip to content

Move MOM_generic_tracer out of MOM6#21

Closed
theresa-cordero wants to merge 2 commits into
NOAA-CEFI-Regional-Ocean-Modeling:dev/cefifrom
theresa-cordero:move_MOM_GenericTrc_module
Closed

Move MOM_generic_tracer out of MOM6#21
theresa-cordero wants to merge 2 commits into
NOAA-CEFI-Regional-Ocean-Modeling:dev/cefifrom
theresa-cordero:move_MOM_GenericTrc_module

Conversation

@theresa-cordero
Copy link
Copy Markdown
Collaborator

Move MOM_generic_tracer out of MOM6 and into the externals as a stub. With this change, users of the generic tracer modules will be requied to add an active version of MOM_generic_tracer to their equivalent of the GFDL_ocean_BGC folder.

This should not impact users running without generic tracers.

This PR is done in two commits, one where the stub has comments and extra use statements which are removed in the second commit.

Theresa Morrison added 2 commits December 16, 2024 10:12
Move MOM_generic_tracer out of MOM6 and into the externals as a stub.
With this change, users of the generic tracer modules will be requied
to add an active version of MOM_generic_tracer to their equivalent of
the GFDL_ocean_BGC folder.

This should not impact users running without generic tracers.
Remove comments and use statements from MOM_generic_tracer stub.
@theresa-cordero
Copy link
Copy Markdown
Collaborator Author

This PR is related to PR 107.

If a version of MOM6 with this change is used with a version of ocean_BGC without the changes in PR 107, then the model will fail to compile.

use g_tracer_utils, only: g_tracer_type

use MOM_ALE_sponge, only : ALE_sponge_CS
use MOM_coms, only : EFP_type
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Remove the trailing space at the end of the line.


use MOM_ALE_sponge, only : ALE_sponge_CS
use MOM_coms, only : EFP_type
use MOM_diag_mediator, only : diag_ctrl
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Same here. a trailing space is detected at the end of the line

use MOM_open_boundary, only : ocean_OBC_type
use MOM_restart, only : MOM_restart_CS
use MOM_sponge, only : sponge_CS
use MOM_time_manager, only : time_type
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Same. Remove the trailing space at the end of the line

@yichengt900
Copy link
Copy Markdown

@theresa-morrison, as you suggested, we can start with our first PR by simply moving MOM_generic_tracer into external. I have created a new branch, dev/cefi_to_gfdl, and you can resubmit the PR to that branch.

@theresa-cordero
Copy link
Copy Markdown
Collaborator Author

Closing this PR for now, a new PR has been opened to the new branch.

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