precice / calculix-adapter

preCICE-adapter for the CSM code CalculiX
GNU General Public License v3.0
51 stars 20 forks source link

Fixed DOF 0 bug & removed useless memcpy #104

Closed boris-martin closed 1 year ago

boris-martin commented 2 years ago

Content

This PR seems to fix https://github.com/precice/calculix-adapter/issues/86 Following a suggestion from Matthias Freimuth, I removed a memcpy statement that seems to fixe the bug. Extensive testing is needed to ensure nothing else is broken. For the record, I am absolutely clueless as to 1) why removing this line fixes the bug 2) what was the purpose of it at first. So, mostly empirical stuff.

In my opinion, three things should be checked:

@KyleDavisSA any idea what's happening here ?

Checklist

On my side:

Fixing the bug?

Non-regression

Memory checks

Nothing done so far.

MakisH commented 1 year ago

@MatthiasFreimuth could you please help us understand this bugfix?

@boris-martin the "Does the partitioned beam work as before ?" box is still unticked. Does this mean you still want to test it, or that you saw issues?

If you say that this already fixed some known cases and it does not break others, let's merge it.

boris-martin commented 1 year ago

I didn't test it yet. But given the good results on other cases I don't expect any issue.