-
Notifications
You must be signed in to change notification settings - Fork 37
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 smoothing to remapping of SSS restoring #883
Conversation
This merge also includes some other minor clean-up and reorganization. The documentation is also updated and cleaned up to reflect these changes.
a5fdaa6
to
4a457cd
Compare
This requires a major reorganization of the step to be more like `remap_topography` after its recent reorganization. Remapping is performed with `mbtempest`. The public function `remap_sss()` has been removed. (It was not mentioned in the docs.)
4a457cd
to
a677010
Compare
TestingI ran the Compare with a similar plot without smoothing: Results are in:
|
@vanroekel, this is now ready for your review. Feel free to review the code (by inspection) but the main review I would like is whether you are good with the outcome. |
@maltrud, @milenaveneziani, @alicebarthel and @irenavankova, you all commented on #872 so I wanted to make sure you're aware of this follow-up work. |
It looks good to me @xylar, thanks for working on this. |
It's something we could think about. It relies on conservative remapping with MOAB and that performs really badly with latitude-longitude grids (because there are so many tiny, overlapping cells near the poles). I had to switch to putting the SSS fields on a cubed-sphere grid here to make the remapping/smoothing to the MPAS mesh performant. I'm not sure how feasible it is to take a similar approach in MPAS-Analysis. |
I see. Thanks for clarifying that part, I had missed it. |
I see I forgot to add that to the PR description! My mistake. It's there now. |
Thinking about this further, I would feel weird about smoothing an "observational" dataset presented in MPAS-Analysis. I think it would be more honest to just mask out the parts we don't trust than to manipulate them. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approving by visual inspection of code and examination of resulting smoothed field.
Thanks @xylar this looks great to me
Thanks @vanroekel! |
Thanks @xylar , this looks great! |
This requires a major reorganization of the step to be more like
remap_topography
after its recent reorganization.Remapping is performed with
mbtempest
. The public functionremap_sss()
has been removed. (It was not mentioned in the docs.)This merge also adds a remapping step to the utility that generates the SSS field from WOA23. The data is remapped to a cubed-sphere grid at ne300 (~10 km) resolution. Remapping from the cubed sphere to MPAS meshes will be much more performant than remapping from the WOA original lat-lon grid, particularly when there is smoothing near the poles as we do here.
The smoothing goes linearly from zero at 70 N to 1000 km at 90 N.
Checklist
Testing
in this PR) any testing that was used to verify the changes