Cleanup of remap_restarts processing of time-invariant parameters in restart files

[gmao-rreichle]

There appears to be obsolete reading of data in remap_restarts processing:

GEOSgcm_GridComp/GEOSagcm_GridComp/GEOSphysics_GridComp/GEOSsurface_GridComp/Utils/mk_restarts/CatchmentRst.F90

Line 258 in 80c7acd

subroutine read_shared_nc4(this, formatter, rc)

Specifically, the time-invariant parameters that are read in subroutine read_shared_nc4() should not be needed.

When Catch[CN] restart file is remapped from one resolution or bcs version to another resolution or bcs version, the time-invariant parameters in the target restart file must come directly from the target bcs. The time-invariant parameters in the source restart file should not be remapped.

The same probably applies to CatchmentCNRst.F90.

See also items 3. and 4. in #618 (comment)

cc: @weiyuan-jiang @biljanaorescanin

[gmao-rreichle]

Addressed in #670. Closing this issue.