Skip to content
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

New candidate for no vertical interp #159

Open
rdPatmore opened this issue Apr 12, 2024 · 3 comments
Open

New candidate for no vertical interp #159

rdPatmore opened this issue Apr 12, 2024 · 3 comments

Comments

@rdPatmore
Copy link
Contributor

rdPatmore commented Apr 12, 2024

@jdha I can see you have created a branch under #158 for adding a feature for by-passing vertical interpolation. I have created separate version in parallel and I wanted to create branch that didn't interfere with yours. This enhancement appears to function well. Instead of placing it in the profile code I have rewritten the zgrv code to handle each case and have added a namelist option to control the feature.

@rdPatmore
Copy link
Contributor Author

I have tested this code. Bounday files and coordinates are created fine, but the model doesn't seem to run past timestep 4 when using these in combination with interpolation on the fly. I am yet to map whether this issue steps from the pybdy code or my application of these bdys to nemo.

@rdPatmore
Copy link
Contributor Author

I have plotted temperature. The fields look sensible but it appears the pybdy has chopped off the bottom - depths now have a maximum of ~1300 m instead of 5000+ m. Also, e3u has been set to nan everywhere, whereas this is not the case for e3t.

The way I have coded this up we set the depth to the same as the src. However, the code still runs through the interpolation. It would be cleaner to also bypass the vertical interpolation. I have been attempting to understand the details of the interpolation in order to do this but I’m not quite there with the understanding.

@rdPatmore
Copy link
Contributor Author

The fix has been made into a pull request #163.

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

No branches or pull requests

1 participant