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

Make conversion to theta optional in spice? #68

Open
bradyrx opened this issue May 7, 2020 · 1 comment
Open

Make conversion to theta optional in spice? #68

bradyrx opened this issue May 7, 2020 · 1 comment

Comments

@bradyrx
Copy link

bradyrx commented May 7, 2020

I'm thinking that the conversion from T to theta in spice should be an optional feature. I imagine it is assumed for observational data that T is in situ T. However, a lot of model output comes out as theta (potential temperature).

Maybe it would be good to add theta=False at the end, then for the following lines, wrap with if not theta:

# FIXME: I'm not sure about this next step.
pt = sw.ptmp(s, t, p)

Also this must be referencing an old version of seawater. sw.ptmp doesn't exist now. I believe it's now temppot.

I'm happy to implement all this if it makes sense and this package is still actively developed.

@ocefpaf
Copy link
Member

ocefpaf commented May 7, 2020

Sure. A PR would be most welcomed! Note that we use the python-seawater and it is still ptmp. There has been no updates as far as I know b/c that is deprecated. (We should all be using gsw's TEOS-10 now;-p)

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