Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

RTTOV12 support initial commit and tested against RTTOV11 version for clear-sky test #17

Closed
wants to merge 2 commits into from

Conversation

david-rundle
Copy link

I've started this pull request to kick off a discussion about what's left to do regarding RTTOV support and to advertise the fact that the branch is here if any interested in the COSP community would like to implement specific features that they would like to use.

There isn't recent reference data for RTTOV in COSP v2 so I had to use some old data from v1.4 from when I did the RTTOV integration. The calls to write out rttov_tbs in the netcdf file seem to be missing (by design?) also. I have left them out for now but can presumably add them back as a result of discussion here. There were some small differences (max <0.5K typically <0.1K) suggesting that the implementation is probably correct but with limited test data it's difficult to say conclusively.

@alejandrobodas
Copy link
Collaborator

Hi @david-rundle
Thanks for this pull request. Currently working from home without access to an RTTOV library. I have built the standard COSP2 driver, without RTTOV support, and run cosp2_test.
The first time I run it, I forgot to set Ltbrttov to .false. and I got the following non-critical errors:

ERROR: COSP input variable: cospgridIN%o3 contains values out of range
ERROR: COSP input variable: cospgridIN%emis_sfc contains values out of range
ERROR: COSP input variable: cospIN%u_sfc contains values out of range
ERROR: COSP input variable: cospIN%v_sfc contains values out of range
ERROR: COSP input variable: cospIN%lat contains values out of range

I guess this is caused by a lack of initialisation of these variables to sensible physical values when RTTOV is called when COSP is built without RTTOV support. They go away when Ltbrttov is set to .false.

I have looked at the changes and I don't see any major problems. I'd prefer to have the output in the standard netcdf files, and remove all the write statements. This will allow regressions tests in future developments.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants