Add Rossby localization documentation#1238
Conversation
travissluka
left a comment
There was a problem hiding this comment.
ugh, who put that Rossby radius localization in there without any associated documentation!?!? Whoever it was, that person no longer deserves to oversee soca!!
travissluka
left a comment
There was a problem hiding this comment.
I'm rescinding my previous approval:
Now that sequential EnKF is merged in we should add a short compatibility note? A user who picks Rossby loc for EAKF will hit a runtime ABORT from ObsLocRossby::computeLocalization(Point3, Point3).
Suggested addition:
Solver compatibility
Rossby localization currently works only with LETKF / GETKF. It is not yet supported with sequential EnKF solvers (e.g. EAKF), which require obs–obs localization. rossby_radius is only defined on the model grid, not at arbitrary obs pairs. Invoking it from a sequential solver aborts at runtime.

Description
All in the title
Checklist