Enable use of `verbose` option in `time_interp_external2` calls from `data_override`

J-Lentz commented 4 months ago

Description In data_override_mod, rather than calling time_interp_external2 subroutines with a hardcoded verbose=.false. parameter, verbose is now controlled by the debug_data_override namelist variable from data_override_nml. Excess spaces before line-continuation ampersands were removed as well.

Fixes #1398

How Has This Been Tested? Builds on the AMD box with Intel compiler version 2024.1.0.

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] Any dependent changes have been merged and published in downstream modules
[x] New check tests, if applicable, are included
[x] make distcheck passes

rem1776 commented 4 months ago

@J-Lentz Just realized this is failing the linter, looks like there's a line length > 120 in data_override.inc

J-Lentz commented 4 months ago

@J-Lentz Just realized this is failing the linter, looks like there's a line length > 120 in data_override.inc

Fixed in 2c32f59f063f31da37cbfbf8b2a81175c275baa8.

NOAA-GFDL / FMS

Enable use of `verbose` option in `time_interp_external2` calls from `data_override` #1516