-
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
Modified ocean/mesh/remap_topography
to allow smoothing
#863
base: main
Are you sure you want to change the base?
Modified ocean/mesh/remap_topography
to allow smoothing
#863
Conversation
descriptor.to_scrip( | ||
'target.scrip.nc', | ||
expandDist=expandDist, | ||
expandFactor=expandFactor, | ||
) |
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.
Before this call, we want to make sure to set descriptor.format = 'NETCDF3_64BITDATA'
(Check this, it's just my best guess on the names of the attribute and its value.)
@bmooremaley, as we discussed, we'll need some way to support a different topography dataset for lower res regression testing. I pulled out the config options into their own config file: |
0475c64
to
1bd3cb8
Compare
The
ocean/mesh/remap_topography
step has been modified to remap combined topography from a cubed sphere source grid to MPAS meshes, instead of a lat-lon source grid. The weights file generation has been switched tombtempest
so that smoothing can be defined in the target MPAS mesh SCRIP file. Smoothing parameters have been added toocean/mesh/remap_topography.cfg
. This new workflow also uses a pre-build source SCRIP file stored in the bathymetry database, in addition to the combined topography file.Checklist
Testing
in this PR) any testing that was used to verify the changes