[PROJ] promoteTo3D in ftUS

[Javier Jimenez Shaw]

On Thu, 21 Oct 2021 at 21:14, Even Rouault <even.rouault at spatialys.com>
wrote:

>
> Le 21/10/2021 à 20:53, Javier Jimenez Shaw a écrit :
>
> Hi
>
> There is currently a method
>
> CRSNNPtr CRS::promoteTo3D(const std::string &newName,
>                           const io::DatabaseContextPtr &dbContext) const
>
> that adds vertical axis in Meters (if there is no one there, etc).
>
> Sometimes (when working with CRS from USA), I need the vertical one in ft
> or ftUS.
>
> promoteTo3D() will create a Geographic3D CRS or a Projected 3D CRS, ie
> with ellipsoidal heights, not orthometric height. Are ellipsoidal heights
> in ft/ftUS really what you need ?
>
>
> There is another function
> CRSNNPtr CRS::promoteTo3D(const std::string &newName,
>                           const io::DatabaseContextPtr &dbContext,
>                           const cs::CoordinateSystemAxisNNPtr
>                               &verticalAxisIfNotAlreadyPresent) const
>
> but it is "PROJ_INTERNAL" (whatever it means)
>
> Just that it restricts a method from being exported in a class that is
> exported. I tend to be conservative of what I expose in the API if I'm not
> sure it will be needed by external users.
>
>
> Would it make sense to have a function (or overload) that specifies the
> unit?
>
> If that's the function you need, just removing PROJ_INTERNAL, moving it in
> the public part and adding doc should be enough.
>
> The CRS::alterCSLinearUnit() method could also be useful.
>
Thanks. alterCSLinearUnit() seems to be a good option.

>
> Thanks.
>
> .___ ._ ..._ .. . ._.  .___ .. __ . _. . __..  ... .... ._ .__
> Entre dos pensamientos racionales
> hay infinitos pensamientos irracionales.
>
>
> _______________________________________________
> PROJ mailing listPROJ at lists.osgeo.orghttps://lists.osgeo.org/mailman/listinfo/proj
>
> -- http://www.spatialys.com
> My software is free, but my time generally not.
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osgeo.org/pipermail/proj/attachments/20211021/5eb70587/attachment.html>