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

CATS version2023 netcdf input variables - tide_currents #370

Open
VikNikolaus opened this issue Dec 9, 2024 · 3 comments
Open

CATS version2023 netcdf input variables - tide_currents #370

VikNikolaus opened this issue Dec 9, 2024 · 3 comments

Comments

@VikNikolaus
Copy link

Hi everyone,

I am new to working with tides and am trying to compute tidal currents for a specific location in the Southern Ocean using the CATS 2008-version2023 netCDF file.

When using pyTMD.compute.tide_currents, I encountered an error because the OTIS.py script appears to be looking for the variables uRe and uIm. However, the version of the CATS model I am using only provides transports (variables named URe and UIm) rather than velocities. I assume the same applies to the meridional component (VRe/VIm).

I’m unsure how to handle this mismatch. Is there a way to adapt pyTMD to work with transport variables, or is additional processing required to convert transports to velocities?

I apologize if this is a basic question, but I’m currently stumped and would appreciate any guidance.

Cheers,
Viktoria

@tsutterley
Copy link
Owner

Hi @VikNikolaus ,
You're absolutely right, the variable names changed from the (pre-release) version I have. The units might also be different. I'll figure out exactly what was done and get it fixed. Sorry about that!

tsutterley added a commit that referenced this issue Dec 10, 2024
feat: add tidal aliasing period function
fix: add latitude and longitude as potential dimension names
tsutterley added a commit that referenced this issue Dec 10, 2024
feat: add tidal aliasing period function
fix: add latitude and longitude as potential dimension names
@tsutterley
Copy link
Owner

Should be fixed in main. Sorry about that. I should have checked for changes upon the release of TMD3.

@VikNikolaus
Copy link
Author

Brilliant, thank you so much!

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