surface flux not masked: bug if isf cavity

[mathiot]

Context

Some surface flux variables are not masked and could affect model results if ice shelf.

Analysis

• This is a cosmetic issue if no ice shelf and fields mask during post-processing (budget computation for example)
• This is a bug if ice shelf cavity opened because if emp (ie precip, snow and ssr in some case), runoff are not 0 over the ice shelf it will affect the ssh beneath the ice shelf.

It affect 3.6_STABLE and the trunk

Fix

mask surface fluxes with tmask(:,:,1):

• zerp and zqrp in sbcssr
• rnf in sbcrnf
• elap, evap, cseep, esre in sbcblk_mfs
• tprecip, sprecip, qns in sbcblk_core
• tprecip, qla_ice, z_qlw, sprecip, zqla, zqsb, zqlw in sbcblk_clio

[mathiot]

In 9567:

Error: Failed to load processor CommitTicketReference

No macro or processor named 'CommitTicketReference' found

[mathiot]

• Fix for version 3.6_stable in r9567
• will be fixed in trunk once dev_merge_2017 will be merged into the trunk

[mathiot]

Ticket has been closed during the commit. Ticket needs to be kept open as not yet done for the trunk.

[mathiot]

Forgot to mention: NO change in SETTE test results

[nicolasmartin]

Replying to mathiot:

Ticket has been closed during the commit. Ticket needs to be kept open as not yet done for the trunk.

This was related to the your commit message, the syntax 'fix #2083' was interpreted as a command to close the ticket. You should use the syntax 'ref #2083' instead when you push a partial commit
See wiki:Developers/WorkingFAQ#commitmessage? for more

[mathiot]

In 9569:

Error: Failed to load processor CommitTicketReference

No macro or processor named 'CommitTicketReference' found

[mathiot]

TRUSTING failed this night (10/05/18) because some restart fields changed in ORCA2LIMPISCES and AMM12.

• In O2LIMPISCES, as expected emp_b, rnf_b, hdiv_n and hdiv_b changed over land. Other restart fields didn't change. TRUSTING for this configuration could be restarted.
• In AMM12, rnf_b changed unexpectedly. It apears, in AMM12, runoff is defined in point (1,1), ie on 'land' point. All other fields are similar. TRUSTING for this configuration could be restarted.

[mathiot]

Yesterday, I forgot to apply mask in sbcflx (AMM12). It is corrected in r9569. There is no change in the results (solver.stat). However, TRUSTING is expected to fail for AMM12 with ' Restarts: ./AMM12_trust_00000576_restart_oce_out 2 record(s) differ ' because qns_b and emp_b will change over land.

[mathiot]

The TRUSTING failed. 2 extra variables (in addition to the ones mentioned above) changed. The masking of surface flux changes also the land value only of sbc_hc_b and tb in AMM12. The new land value is 0.0. Previously in AMM12, the land value of tb and sbc_hc_b was not 0.0 because surface flux was not 0.0. All looks normal. TRUSTING for this configuration could be restarted.

[nicolasmartin]

Replying to mathiot:

The TRUSTING failed. 2 extra variables (in addition to the ones mentioned above) changed. The masking of surface flux changes also the land value only of sbc_hc_b and tb in AMM12. The new land value is 0.0. Previously in AMM12, the land value of tb and sbc_hc_b was not 0.0 because surface flux was not 0.0. All looks normal. TRUSTING for this configuration could be restarted.

Thanks for the clarification.
I will update the benchmark results with the last commit.

[mathiot]

In 9727:

Error: Failed to load processor CommitTicketReference

No macro or processor named 'CommitTicketReference' found

[mathiot]

Fix in r9727 (trunk) and r9567 (3.6_stable)