You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
According to the TEOS-10 documentation for gsw_z_from_p [link], geo_strf_dyn_height, sea_surface_geopotential should be optional arguments, but in this package, they are mandatory.
I believe I was able to fix this by adding the function definition on a local version of GibbsSeaWater.jl function gsw_z_from_p(p, lat) ccall((:gsw_z_from_p, libgswteos), Cdouble, (Cdouble, Cdouble), p, lat) end
in the gen_gswteos10.jl file. Not sure if there is some issue with this solution, but happy to create a pull request if it would be useful.
Thanks for your suggestion @haakon-e.
Thanks for your comment @b-r-hamilton.
Actually, I plan to update them together with binary upgrades (but BinaryBuilding system is down now, please wait).
According to the TEOS-10 documentation for
gsw_z_from_p
[link],geo_strf_dyn_height
,sea_surface_geopotential
should be optional arguments, but in this package, they are mandatory.GibbsSeaWater.jl/src/gen_gswteos10.jl
Lines 733 to 735 in fe9a7cd
The text was updated successfully, but these errors were encountered: