So a few days ago I’d suggested I would take on refactoring BoundingVolumes to remove the overhead of extending Trimesh. Before I launch into designing that change, a few questions…
- Does anyone have an objection to turning BoundingVolume into an abstract class (it’s currently an interface.) This way we can move some code up into that rather than repeating it in each bounding. Originally it was an abstract class (before boundings could be visualized)
- I’m planning on adding a Trimesh field to BoundingVolume that would be lazily set on the first call to draw bounds on a given item. Not having to update certain things each frame in Trimesh should make life better for anyone not visualizing bounds. Any differing views on that?
- I’m going to look at either adding in the suggestions made on plane checks or optimizing what we have.
- Any other optimization ideas out there?
I’ll start in on this probably Monday afternoon, so give me your thoughts by then. Once I’ve got the design together from your comments, I’ll post on the Code Review forum. Thanks!