Skip to content

Varobs cx xios fix 2 - #302

Closed
Jonas Ganderton (mo-jonasganderton) wants to merge 3 commits into
developfrom
feature/varobs_cx_xios_fix_2
Closed

Jonas Ganderton (mo-jonasganderton) wants to merge 3 commits into
developfrom
feature/varobs_cx_xios_fix_2

Conversation

@mo-jonasganderton

Copy link
Copy Markdown
Contributor

Duplicate of #300, seeing if this allow merging instead.

Copied from #300:

Summary

Enables the Varobs Writer and CX Writer filters to work when XIOS is used in server mode.
Done by switching from using mpl_comm_world to mpi_group in the relevant filters.

Testing

mo-bundle tests

I've tested malak kgo_glu_jopa_jada using XIOS in server mode, this results in KGO changes for JOPA and JADA. This is not the default behaviour, but demonstrates this works should we want to change.
This was done using a couple of custom branches - these are not designed for merging, primarily because I've taken the 4dvar iodef for testing purposes.

  • Uses custom jjdocs branch - feature/test_xios
    • New configuration.nml for C48-C192 with XIOS
    • jopa_process adapted to use XIOS server mode
    • XIOS replacing monio in yamls
    • Extra fields to field def
  • Uses custom malak changes to make use of the jjdocs changes

A few ordinary malak and sith runs with the build but no further changes.

Resolves

@mo-jonasganderton

Copy link
Copy Markdown
Contributor Author

Looks like this has the same "Cannot update this protected ref." Will close in favour of the original.

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