ESCOMP / POP2-CESM

Parallel Ocean Program (POP2) in CESM
http://www.cesm.ucar.edu/models/cesm2/ocean/
4 stars 24 forks source link

dont call logging again #53

Closed jedwards4b closed 3 years ago

jedwards4b commented 3 years ago

Description of changes:

Remove a redundant call to set_component_logging

Testing: ERS_Vnuopc.T62_g37.G.cheyenne_gnu.pop-cice

Test case/suite: Test status: bit for bit,

Fixes [POP2 Github issue #]

User interface (namelist or namelist defaults) changes?

jedwards4b commented 3 years ago

Now I'm confused - I removed this because it was present twice in InitializeRealize() but I don't see the repeat now. Let me retest this, maybe it isn't needed.

mnlevy1981 commented 3 years ago

@jedwards4b isn't it this block of code in InitializeAdvertise()?

https://github.com/ESCOMP/POP2-CESM/blob/578cf7b8b6999a383584d5e903401bcaa93e45da/drivers/nuopc/ocn_comp_nuopc.F90#L228-L235