tigl icon indicating copy to clipboard operation
tigl copied to clipboard

False positives in check, if wing cells can be trimmed rather than cut.

Open joergbrech opened this issue 2 years ago • 2 comments

TiGL tries to determine if the geometry of a wing cell can be calculated by trimming a surface rather than cutting it with Boolean operations for a better performance. Currently, this check often has false positives and the trimmed result is wrong. The trimming is currently disabled in https://github.com/DLR-SC/tigl/commit/da2d269c6e943f7b9a2376e10babe2f36b56469d and we should either remove the check completely or change it to be more reliable.

This is related to https://github.com/DLR-SC/tigl/commit/da2d269c6e943f7b9a2376e10babe2f36b56469d and #865.

joergbrech avatar Dec 21 '21 11:12 joergbrech

Currently I would prefer to remove the checks, because the current version leads to wrong cell shapes.

condoratberlin avatar Mar 08 '22 11:03 condoratberlin

@condoratberlin thanks for the feedback. I will prepare a patch release of TiGL either this or next week where the checks are disabled. I will keep this issue open for documentation purposes. If there are no further issues with the disabled tests, we can remove the checks completely at a later stage (which would just be a refactoring of the source code, without any behavioral changes).

joergbrech avatar Mar 08 '22 11:03 joergbrech