Motification of doc for C-grid remapping

[JFLemieux73]

For detailed information about submitting Pull Requests (PRs) to the CICE-Consortium,
please refer to: https://github.com/CICE-Consortium/About-Us/wiki/Resource-Index#information-for-developers

PR checklist

• Short (1 sentence) summary of your PR:
  Minor modifications to the documentation for C-grid remapping and edge flux adjustment method (EFA).
• Developer(s):
  @JFLemieux73
• Suggest PR reviewers from list in the column to the right. @eclare108213, @apcraig
• Please copy the PR test results link or provide a summary of testing completed below.
  BFB for B and C-grid gbox80 tests (with ktranposrt=kridge=1)
• How much do the PR code changes differ from the unmodified code?
  • bit for bit
  • different at roundoff level
  • more substantial
• Does this PR create or have dependencies on Icepack or any other models?
  • Yes
  • No
• Does this PR update the Icepack submodule? If so, the Icepack submodule must point to a hash on Icepack's main branch.
  • Yes
  • No
• Does this PR add any new test cases?
  • Yes
  • No
• Is the documentation being updated? ("Documentation" includes information on the wiki or in the .rst files from doc/source/, which are used to create the online technical docs at https://readthedocs.org/projects/cice-consortium-cice/. A test build of the technical docs will be performed as part of the PR testing.)
  • Yes
  • No, does the documentation need to be updated at a later time?
    • Yes
    • No
• Please document the changes in detail, including why the changes are made. This will become part of the PR commit log.
  People from SI3 are interested in our C-grid remapping and edge flux adjustment method (EFA). They had a look at the code and the documentation. It was not clear in the code (and the doc) that the EFA method is used when setting l_fixed_area=.true. I have changed the name l_fixed_area to l_EFA, modified the doc and added the reference for the C-grid paper.

[apcraig]

It looks like the code uses l_EFA while the documentation suggests the variable name is l_EFA_area. Should these be consistent?

[JFLemieux73]

It looks like the code uses l_EFA while the documentation suggests the variable name is l_EFA_area. Should these be consistent?

Thanks @apcraig. I just modified the doc.

[eclare108213]

I'm not crazy about l_EFA, would prefer l_edge_flux_adj to be more immediately recognizable. But okay. Thanks for clarifying this parameter for users!

[JFLemieux73]

It is true that it is more easily recognizable. I will change it. No problem.

[JFLemieux73]

I just changed the name of the variable. I redid the gbox80 tests and it is BFB for both B and C-grid (with ktransport=kridge=1). Thanks for your help!

[eclare108213]

Thank you for fixing that!