Skip to content
This repository has been archived by the owner on Jul 13, 2021. It is now read-only.

Overload convert_arguments for LineString #479

Closed
wants to merge 4 commits into from

Conversation

Sov-trotter
Copy link
Contributor

@Sov-trotter Sov-trotter commented Jul 23, 2020

This is the very first attempt at supporting GeometryBasics geometries based on MakieOrg/Makie.jl#660. Is there any better method to get to points? A set of decompose/reinterpret methods?

@Sov-trotter Sov-trotter marked this pull request as draft July 23, 2020 11:11
src/conversions.jl Outdated Show resolved Hide resolved
@Sov-trotter
Copy link
Contributor Author

Also I was wondering, what is the correct way to support a vector of defined geometries. One way is iterating manually, but how can that be achieved via convert_arguements. I tried writing one for the above LineString method but that only returned the first LineString.

@Sov-trotter
Copy link
Contributor Author

Sov-trotter commented Aug 8, 2020

@SimonDanisch @asinghvi17 can you take a look?

@Sov-trotter
Copy link
Contributor Author

decompose(PB, LineString) depends on JuliaGeometry/GeometryBasics.jl#73

@Sov-trotter
Copy link
Contributor Author

What changes might go in here, as we are moving towards JuliaGeometry/GeometryBasics.jl#80 in GeometryBasics?
Will the decompose method do the work for us?

@SimonDanisch
Copy link
Member

continued in: #489

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants