Pheidegger cfsdat cell centroid

Patrick Heideggerrequested to merge
pheidegger_cfsdat_cellCentroid into master

Description

Fix some bugs in cfsDat for aeroacoustics and cellCentroid interpolation:

  • A segmentation fault occurres due to a wrong initialization of a counter.
  • Additionally, I discovered that assigning a dof to treat a vectorial quantity does not have an effect and stores everything into the x component of the vector. The dof tag is used to store a scalar quantity from ensight, which can only handle scalars, into an axis of the corresponding vectorial quantity in cfs. This has, however, no effect in my example. Usage, e.g.,:
<variable CFSVarName="meanFluidMechVelocity" EnsightVarName="MappedMean_vMonitor" static="yes" dof="y"/>
  • Also, the tag static appears to have no effect.

Testsuite: Testsuite!120 (closed)

(Summarise the changes introduced in this merge request)

Check before assigning to a maintainer for review

  • The history is clean (squashed into meaninful commits and commit messages)
  • New Code is well-documented and understandable
  • In case of new functionality, there is a test case (also mentioned in the description)
  • Every commit passes the pipeline
  • There are no new build warnings on cdash/pipeline on any build

Maintainer checks before merge

  • Review is approved, and all comments are resolved
  • New tests are actually running (e.g. check if they appear on CDash)
Edited by Dominik Mayrhofer

Merge request reports