Generalize vertices type in VPolytope

[mforets]

It should be:

struct VPolytope{N, VT<:AbstractVector{N}} <: AbstractPolytope{N}
    vertices::Vector{VT}
end

[schillic]

Why restricted it Vector? I would just add another type parameter there.

[mforets]

The vector of abstract vectors is general enough. This is the recommended memory layout for handling many static arrays (https://juliaarrays.github.io/StaticArrays.jl/latest/pages/api/#Arrays-of-static-arrays-1).

[schillic]

StaticArrays is a good keyword. What about polygons with 5 vertices? This should be exploitable.

[mforets]

Of course, there can be applications that are better off with another outer array type, but none that i'm aware. We can always add a new type to handle further specializations.

[schillic]

It does not hurt to add the type parameter now. It hurts to change it later again because that is always a breaking change.