Skip to content

Commit

Permalink
Remove ambiguity and implement getgeom, ngeom for Wrappers.Point
Browse files Browse the repository at this point in the history
  • Loading branch information
asinghvi17 committed Oct 23, 2024
1 parent e6cacdc commit c7a4a1e
Showing 1 changed file with 4 additions and 0 deletions.
4 changes: 4 additions & 0 deletions src/wrappers.jl
Original file line number Diff line number Diff line change
Expand Up @@ -318,6 +318,10 @@ convert(::Type{Point}, ::PointTrait, geom) = Point(geom)
convert(::Type{Point}, ::PointTrait, geom::Point) = geom
extent(trait::PointTrait, geom::Point) = extent(trait, parent(geom))

ngeom(::PointTrait, geom::Point) = 0
getgeom(::PointTrait, geom::Point) = nothing
getgeom(::PointTrait, geom::Point, i) = nothing

Check warning on line 323 in src/wrappers.jl

View check run for this annotation

Codecov / codecov/patch

src/wrappers.jl#L321-L323

Added lines #L321 - L323 were not covered by tests

x(trait::PointTrait, geom::Point) = x(trait, parent(geom))
y(trait::PointTrait, geom::Point) = y(trait, parent(geom))
z(trait::PointTrait, geom::Point{true}) = z(trait, parent(geom))
Expand Down

0 comments on commit c7a4a1e

Please sign in to comment.