Kyle Beggs
Kyle Beggs
> It's true that this is a performance bug, but I'm also tempted to say "don't do that" :-) Even though we do allow mutable field vectors I personally can't...
We should be able to refactor this function into just looping through the triangles in the tetrahedron and calling `intersection(ray, triangle)` now.
Yeah this should be strait forward now. I'll see if I can get to it today, if not it'll be in 1.5 weeks as im traveling
If the ray intersects multiple triangles in the tetrahedron, should we just be returning a vector of the intersections?
I'm not familiar with that
I see. This multi(geoms) object already exist or i need to create it? I haven't seen any type like that
We would need to make changes to `Multi` to make this work. Currently, it only supports `PointOrGeometry`. Either we need to loosen that type restriction or create a new type...
We do allow mixed collections, the issue is that the return type is an `Intersection` which is not a point or geometry. We need to add `Intersection` to the `Union{Point{Dim,T},Geometry{Dim,T}}`...
I think what you have in mind is inserting the geometry from the intersections into `Multi`? I was thinking we really should be wrapping `Intersection` into some kind of `Multi`...
Hi @juliohm, sorry I have been really busy with a few projects of my own and this issue isn't needed for my progress on that. I'm going to try to...