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

Add cx namelist for UKV surface #199

Merged
merged 6 commits into from
Apr 17, 2024
Merged

Add cx namelist for UKV surface #199

merged 6 commits into from
Apr 17, 2024

Conversation

ReubenHill
Copy link
Contributor

This adds a cx namelist for UKV surface processing. This is almost identical to the global cx namelist (#152), except that the cx fields are explicitly given as suggested in #155 .

The fields were chosen by inspecting the ukv surface OPS .varobs files to see which fields are output. Since the present status of the UKV surface processing uses the same fields as the global processing (visibility has yet to be ported), those which are removed in #152 are removed here. I noted that not all of them have been ported to JOPA, but found that the VAR task in sith for both JOPA and OPS gave equivalent outputs when buddy check and thinning are switched off (i.e. within double vs single precision floating point arithmetic error.)

With thinning and buddy check turned back on, one yields the following analysis increment difference with this version of sith, this JJAUX and this jjdocs which contains the surface processing yaml. Cylc review logs here.
Analysis_increments_Hist_lev_01

I expect that visibility will need to be added back in when that is ported.

Copy link
Collaborator

@ctgh ctgh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for adding this. I like the table in the namelist file.

@ctgh ctgh requested a review from mikecooke77 April 16, 2024 10:42
@mikecooke77 mikecooke77 merged commit 0844270 into develop Apr 17, 2024
6 of 7 checks passed
@mikecooke77 mikecooke77 deleted the feature/ukv-surface branch April 17, 2024 09:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants