plot output from box tests

[eclare108213]

• Short (1 sentence) summary of your PR:
  script modifications for plotting output from box tests
• Developer(s):
  @eclare108213
• Suggest PR reviewers from list in the column to the right.
• Please copy the PR test results link or provide a summary of testing completed below.
  ENTER INFORMATION HERE
• 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.

Added a python script to make simple 2-dimensional plots of output from box tests, in conjunction with the slotted cylinder tests shown in #1072. The new python script closely mimics the original one, which only made plots on global lat/lon grids. I also altered the filenames and logic in ciceplots.csh. The current version here shows what was needed for my box plots in the user-definedfiles, notes, and fstr data. I'd be happy to comment mine out in favor of the original, since (I think) it's used to create plots when we make a release, but there must be a better way to do this. Suggestions are most welcome...

I added the advection namelist option to set_nml.boxslotcyl, since that test is aimed at advection and I wanted to check upwind too. The code itself should still be BFB, including slotted cylinder tests.

[apcraig]

I would move gridtype and plottimeseries (lines 56-62) settings up to the top of the script. Then I would add an if block after that for global or box to set the case, casedir, files, notes, and fstrs. And could we set the default to 'global' and 'true'. These are needed for making release plots, I think it would be good to have that be the default. Otherwise, this seems fine overall. Are you happy with the box plots or need any help otherwise?

[eclare108213]

I rearranged ciceplots.csh so that the settings for both global and box grids are available, and set the defaults for the global grids. In my setup, ${MODULESHOME} is not defined (should it be?), so I still have to comment that out. Does that source command need to be at the top, or can I also move it into the 'global' block?

The only remaining weirdness for box plots is that timeseries.py labels the box output as 'Arctic' (the code uses NH longitudes), and the 'Antarctic' lines are all zero. That's not optimal, but for this purpose it's good enough in my opinion, rather than adding yet another script specific for labeling box timeseries.

@apcraig please, would you test this version in your regular setup to make sure I haven't broken something?