Skip to content

Make conversion to theta optional in spice? #68

@bradyrx

Description

@bradyrx

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.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions