Repository navigation
doc: add rttov sensor file namelist item for rttov13 - #1202
hkershaw-brown wants to merge 3 commits into
Conversation
| .. code-block:: text | ||
|
|
||
| &obs_def_rttov_nml | ||
| rttov_sensor_db_file = 'unspecified' ! Set to rttov_sensor_db.csv, example provided in DART/observations/forward_operators/rttov_sensor_db.csv |
There was a problem hiding this comment.
I don't think this should be 'unspecified'. I think it would be better to have the this be 'rttov_sensor_db.csv' like it is in the docs for the 12 version.
Then overall, I think it would be better to actually change the default value in obs_def_rttov_mod.f90 and obs_def_rttov13_mod.f90 from 'unspecified' to 'rttov_sensor_db.csv'. We could then maybe have a more helpful error message in subroutine read_sensor_db_file
There was a problem hiding this comment.
I totally agree with this. Let's loop in Brett on the rttov default filename option before changing the code.
There was a problem hiding this comment.
Suggested changes all seem reasonable to me.
There was a problem hiding this comment.
ok great @braczka I'll go ahead and make the code changes.
note on "default" file - not the default in the namelist, the file provided by dart fixes #1197
2068be0 to
fbfc3a0
Compare
Description:
Please provide a summary of the change and include any relevant motivation and context.
'rttov_sensor_db.csv'
Fixes issue
Fixes #1197
Types of changes
Documentation changes needed?
Tests
Please describe any tests you ran to verify your changes.
Spellcheck of coefficient: https://nwp-saf.eumetsat.int/site/software/rttov/download/coefficients/coefficient-download/
Built documentation locally
Checklist for merging
Checklist for release
Testing Datasets