Johannes Blühdorn
Johannes Blühdorn
Target branch is `master`, you probably wanted it to be `develop`?
> @jblueh are the tsan tests using a special version of CoDi? > > ``` > Note: switching to 'refs/pull/2158/merge'. > > You are in 'detached HEAD' state. You can...
@alecappiello develop is updated with a fix for the thread sanitizer issue, it should fix your CI pipeline once you merge it into your branch
@joshkellyjak The memory access violation described [here](https://github.com/su2code/SU2/pull/2246#issuecomment-2034404720) does not occur unconditionally, it depends on the test case. The iterations of the inner loop for `jSpan == 0` and `jSpan ==...
Thanks @joshkellyjak and @pcarruscag for your remarks!
Interface between the moving and stationary zones generates wrong results as the moving zone rotates
I just found a heap buffer overflow in the interface code, mentioning it here in case it is related https://github.com/su2code/SU2/pull/2246#issuecomment-2034404720
Interface between the moving and stationary zones generates wrong results as the moving zone rotates
> I just found a heap buffer overflow in the interface code, mentioning it here in case it is related [#2246 (comment)](https://github.com/su2code/SU2/pull/2246#issuecomment-2034404720) I ran your test case with the fixes...