openfoam-adapter icon indicating copy to clipboard operation
openfoam-adapter copied to clipboard

Incomplete checks for supported data location types

Open MakisH opened this issue 2 years ago • 0 comments

https://github.com/precice/openfoam-adapter/pull/206 adds some sanity checks on whether the user is asking for a valid combination of data type (displacement, force, ...) and location (face centers / face nodes).

However, this does not currently cover cases in which we can only read but not write (and the reverse), asumming that the user knows what she is doing. This is a bit dangerous, because it creates a false impression of enough checks for every case. However, it was necessary because the rest of the code also does not nicely handle this distinction between reading and writing.

We could extend/redesign the code to make such checks easier.

Motivated by https://github.com/precice/openfoam-adapter/pull/206#discussion_r752455916.

MakisH avatar Nov 30 '21 21:11 MakisH