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

Adding vgrid support for PySCHISM #87

Open
saeed-moghimi-noaa opened this issue Jul 17, 2023 · 7 comments
Open

Adding vgrid support for PySCHISM #87

saeed-moghimi-noaa opened this issue Jul 17, 2023 · 7 comments

Comments

@saeed-moghimi-noaa
Copy link

@cuill @BahramKhazaei-NOAA
A place holder to follow up with Portugal team for their python implementation and more advanced methods

@BahramKhazaei-NOAA
Copy link

Let's consider incorporating LSC2 vgrid generation in pyschism.

@cuill Please provide us additional resources you have regarding that.

@cuill
Copy link
Member

cuill commented Jul 19, 2023

@saeed-moghimi-noaa
Copy link
Author

@BahramKhazaei-NOAA @cuill @SorooshMani-NOAA

Would you test the script and suggest how we may implement this within the pyschim framework?

Thanks,
-Saeed

@BahramKhazaei-NOAA
Copy link

@BahramKhazaei-NOAA @cuill @SorooshMani-NOAA

Would you test the script and suggest how we may implement this within the pyschim framework?

Thanks, -Saeed
@cuill This is very good to start with. Thanks for sharing.
@saeed-moghimi-noaa I will test the script shortly.

@felicio93
Copy link
Contributor

I tested the script provided and it works for cases where hsm and nv increase constantly by +1.

However if I tried to use a custom nv (e.g., [2,4,6,8,11,14,17,20,20]) it would result in NaN sigma values, please see the vgrid.in below:
vgrid_py_nan

To solve this I had to modify the script, you can see the jupyter notebook here.

I also changed the way the file is written, so it matches the LSC2 format.

I was able to run SCHISM with the vgrid I created.

@felicio93
Copy link
Contributor

@josephzhang8 @feiye-vims

We here at OCS want to push for implementing a lsc2-based vgrid support for PySCHISM.

What do you recommend? We want to make sure you guy are not already developing a solution so we don't duplicate work.

@saeed-moghimi-noaa, feel free to add here if I am missing anything,

@feiye-vims
Copy link
Member

We're not developing a LSC2 vgrid support under pyschism.

Please go ahead.

Thanks!
Fei

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

5 participants