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
Support for things like Pointer, LineStringer, etc. Should be dropped from the general code base. They were added for interoperability, but support has been sparse and adds an extra cognitive load in switch x.(type) using code.
For interoperability, I propose a convert or interop package with contains the interfaces and and has a ToGeom function. A readme would also have an example on how to write a FromGeom function, and perhaps a FromGeomer interface which contains the necessary functions to convert from the geom types.
The text was updated successfully, but these errors were encountered:
Support for things like
Pointer
,LineStringer
, etc. Should be dropped from the general code base. They were added for interoperability, but support has been sparse and adds an extra cognitive load inswitch x.(type)
using code.For interoperability, I propose a
convert
orinterop
package with contains the interfaces and and has aToGeom
function. A readme would also have an example on how to write aFromGeom
function, and perhaps aFromGeomer
interface which contains the necessary functions to convert from the geom types.The text was updated successfully, but these errors were encountered: