-
Notifications
You must be signed in to change notification settings - Fork 15
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
gsw_ct_freezing and gsw_t_freezing switched to "exact" #10
Conversation
This is the minimal change; if backwards compatibility is not required, we could rename the "exact" versions instead.
@richardsc, @dankelley would you look at this, please? From your perspective, is there a need to retain the redundant "exact" function names? Are R users using them? |
We are at the early stages with the R package. I don't think there are too many people using it, beyond as a way to get CT and SA. The decision we made was to have functions named similarly to those of Matlab, as a way to help users to make a transition. So, for example, the R function I think what you're saying, Eric, is that you are going to make the same choice in your python work. I've not checked the details of other functions. Basically, I'm quite neutral on what things are called in the C code, because I can do a To be honest, I'm a bit out of touch on who is in charge of which sets of code. I'd be in favour of a half-hour web-video meeting with all involved (hard, with the timezones), to get an idea for distributing labour amongst participants. I'd also like to get an idea on the versioning scheme; the teos-10 webpages made me think that the C was for 3.05, but the code is partly 3.03, as far as I can tell. The reason the versioning matters is that I'd like to name the R code as e.g. 3.05-1 for a first draft, 3.05-2 for a second, etc ... but if the original (matlab) changes, I'd jump to 3.06-1 etc. I want users to know (a) the version of the base algorithms and (b) the version of my wrapper. |
@dankelley, Versioning and a conference: yes, I agree 100%. The versioning has been murky and incomplete, and it would be good to have some better communication among people interested in any or all of the TEOS language implementations. I'm not sure whether a video conference is feasible, spanning eastern Canada to Australia, but we need to do something. |
Yes, the distance can be a problem ... see http://www.cbc.ca/radio/asithappens/as-it-happens-thursday-edition-1.4047704/a-tale-of-two-sydneys-dutch-teen-tries-to-visit-australia-but-ends-up-in-nova-scotia-1.4047709 ... |
@hylandg Thanks for the merge. I decided to go ahead with a thorough cleanup of this part of the API, so I will have a PR for that momentarily. |
The version number. (first_number.seond_number.third_number) The first number indicated which version of the look up table is used to make salinity, there have been 3 version to date. I doubt we will see a 4th version for a long time. The second number relates to significant changes to the calculated values such as specific volume etc. or if we introduce a whole new section. The third number is for small bug fixes. |
This is the minimal change; if backwards compatibility is
not required, we could rename the "exact" versions instead.
This partly addresses #9; see also TEOS-10/GSW-Fortran#4.