Skip to content

[surface model] Fix boolean transformation

Severin Diederichs requested to merge fix_boolean_transformation into surface_model

MR !1072 (merged) enabled that the first local transformation is the identity. This allowed to reduce the number of total transformations since all surfaces with the identity transformation didn't need to recreate their own transformation. However, for boolean solids the assigned local transformation is later altered. That led to the issue that the fLocalTrans[0] was altered and was not the identity anymore, thus breaking the transformations and causing point location errors.

This MR fixes this by using the index 0 for the identity only for nonboolean solids such that fLocalTrans[0] is not altered. This fixed all observed point location errors

Edited by Severin Diederichs

Merge request reports