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

Definition File for HAMTIDE constituents #179

Closed
martin-rdz opened this issue Apr 1, 2023 · 3 comments
Closed

Definition File for HAMTIDE constituents #179

martin-rdz opened this issue Apr 1, 2023 · 3 comments

Comments

@martin-rdz
Copy link

Hi everybody,

I want to use the constituents provided by hamtide within pyTMD.
Basically, the data is provided as separate netcdf file for each constituent and split up into amplitude and current information.
Can someone please provide me with some guidance on how to setup the Definition Files?

Thank a lot in advance!
-Martin

@tsutterley
Copy link
Owner

Hi @martin-rdz,
I was hoping to have a quick reply to get you going with the definition files. However, it looks like HAMTIDE has a slightly different convention in terms of netCDF variable names. So I added HAMTIDE11 as a supported model within PR #180. This uses the FES.io functions, but adds the new variable names. Tested the height outputs using the forecast notebook, and everything seemed reasonable compared with other models. Hope this helps.
Cheers,
-ts

@tsutterley
Copy link
Owner

Alright, #180 is merged. I'll put together a release later this week

@martin-rdz
Copy link
Author

Sorry, took a while before I could have a look.
Works pretty fine.
Thanks a lot for the quick addition!

Best,
martin

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

2 participants