"Tensor element quadratic 3D get_sub_bounds() returns full bounds, don't use."
Is this being used in bvh construction? I feel like this can be implemented.
Yes it's possible to implement get_sub_bounds() for the tensor specializations, but I didn't want to add another dependency on MultiVec, and besides, get_sub_bounds() is not implemented for the simplex specializations.
get_sub_bounds() is not being used in bvh construction. In bvh construction the DetachedElement makes it possible to get the actual coefficients of a split element, so get_bounds() is used instead.
The only place get_sub_bounds() is actually used, is in contour.cpp. It could be removed if sub_bounds were stored in the augmented bvh. Note that to precompute the sub_bounds, we would need to replay the splits done for the mesh bvh and use DetachedElement again.
The string 'get_sub_bounds' does also appear in some code that is not being used.
mclarsen
commented
4 years ago 
Yes it's possible to implement get_sub_bounds() for the tensor specializations, but I didn't want to add another dependency on MultiVec, and besides, get_sub_bounds() is not implemented for the simplex specializations.
get_sub_bounds() is not being used in bvh construction. In bvh construction the DetachedElement makes it possible to get the actual coefficients of a split element, so get_bounds() is used instead.
The only place get_sub_bounds() is actually used, is in contour.cpp. It could be removed if sub_bounds were stored in the augmented bvh. Note that to precompute the sub_bounds, we would need to replay the splits done for the mesh bvh and use DetachedElement again.
The string 'get_sub_bounds' does also appear in some code that is not being used.