search

tumcms / Open-Infra-Platform

This is the official repository of the open-source Open Infra Platform software (as of April 2020).
Other
47 stars 21 forks source link

Slow calculation time for IfcSine and IfcCosine #560

Open Elvira2227 opened 1 year ago

Elvira2227 commented 1 year ago

Describe the bug Calculation of IfcSine and IfcCosine takes a long time.

To Reproduce Steps to reproduce the behavior:

  1. Go to CurveConverter.h
  2. Scroll down to the line 2693 for IfcSine or to the line 2748 for IfcCosine
  3. Set an additional breakpoint if necessary
  4. Wait for the end of the calculation

CMake configuration If applicable, list any special configurations you have selected in CMake.

Additional context On my machine it took approximately 9 minutes.

pjanck commented 1 year ago

Well that's just how the calculation is organized. Improvements are definitely possible. But I would argue that this currently has a low priority.