Skip to content

Add SternaMWS and EPSMWS to opsinputs#239

Merged
ctgh merged 3 commits intodevelopfrom
feature/addSternaMWS
Apr 22, 2025
Merged

Add SternaMWS and EPSMWS to opsinputs#239
ctgh merged 3 commits intodevelopfrom
feature/addSternaMWS

Conversation

@brettcandy
Copy link
Contributor

@brettcandy brettcandy commented Apr 9, 2025

This is a PR to allow the creation of varobs files using the opsinputs filters VarObs Writer and Varcx Writer for two new obsgroups, SternaMWS and EPSMWS Both are conventional MW sounders. As such there are no new varfields aor cx fields required. It is just small updates to allow the filters to recognise these new datasets

I have set these upto have the following obsgroups

  • 70 SternaMWS
  • 71 EPSMWS

@brettcandy brettcandy changed the title first working version with new ctest Add SternaMWS and EPSMWS to opsinputs Apr 9, 2025
@brettcandy
Copy link
Contributor Author

Tests

i have created a new ctest for varobs writing for SternaMWS.

i have also taken the code updates here and sucessfully built mo-bundle on EX machine
output here

We have SternaMWS data in the building and so i ran a version of sith with these new data and successfully wrote Varobs/Varcx files output here

@brettcandy
Copy link
Contributor Author

Viewing the new varobs file

I have used the varpy functionality uvea to interogate the varobs file. The records are as expecteed. E.g. here is the satellite viewing angle field 31
satview

@brettcandy
Copy link
Contributor Author

Ready to review - thanks

@ctgh
Copy link
Collaborator

ctgh commented Apr 14, 2025

@brettcandy For my information are you planning to add namelist files for EPSMWS as well?

@mikecooke77
Copy link
Collaborator

Looks good to me and its already got two reviews. If you are happy that no more changes are needed please add the "ready to merge" flag.

@brettcandy
Copy link
Contributor Author

@brettcandy For my information are you planning to add namelist files for EPSMWS as well?

i think i will add these namelists once we have actual data - in case something changes.

@ctgh ctgh merged commit af99128 into develop Apr 22, 2025
7 of 9 checks passed
@ctgh ctgh deleted the feature/addSternaMWS branch April 22, 2025 09:15
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.

5 participants