SU2 icon indicating copy to clipboard operation
SU2 copied to clipboard

Aachen Turbine Test Case

Open jblueh opened this issue 9 months ago • 1 comments

The Aachen turbine test case added in #2158 needs another look, for two reasons.

  1. The test case is subject to a yet unfixed memory access violation, as detected in the address sanitizer tests in #2246. We explored a fix in c97610b02bc1281eb766847e7ef5ed7ac451dfb9, that remedies the address sanitzer detections but also changes the test results notably. It is unclear yet whether this is the correct way to fix it.
  2. @bigfooted observed non-deterministic behaviour in #2260. This might or might not be related to the memory access violation.

As soon as the memory access violation is fixed, address sanitizer testing can be re-enabled for the Aachen turbine test case (it was disabled in 0f3fc3ec900f1518c0c3ed26c3e971ae27fb974c).

@alecappiello We can continue the discussion from #2246 here.

jblueh avatar May 08 '24 11:05 jblueh