Just documenting this issue I highlighted in the code a while back:
|
! TODO(Sean): we should not be setting namelist parameters in the following if |
|
! block - all options are all configurable via the namelist file and is |
|
! unclear that these options are being overwritten. A better approach would be |
|
! to error for bad combinations of namelist parameters. |
|
IF (icycle > CASAONLY_ICYCLE_MIN) THEN |
|
icycle = icycle - CASAONLY_ICYCLE_MIN |
|
CASAONLY = .TRUE. |
|
CABLE_USER%CASA_DUMP_READ = .TRUE. |
|
CABLE_USER%CASA_DUMP_WRITE = .FALSE. |
|
ELSE IF (icycle == 0) THEN |
|
CABLE_USER%CASA_DUMP_READ = .FALSE. |
|
spincasa = .FALSE. |
|
CABLE_USER%CALL_POP = .FALSE. |
|
END IF |
Setting icycle to a value greater than 10 is redundant, should we remove it?
