search

Nek5000 / nekRS

our next generation fast and scalable CFD code
https://nek5000.mcs.anl.gov/
Other
289 stars 76 forks source link

Enable new tavg plugin for CHT use #495

Closed joneuhauser closed 1 year ago

joneuhauser commented 1 year ago

Since v23, the tavg module allows to specify the fields (and pointwise products) to be averaged. However the kernel always operated on the size of the vmesh only, i.e. when passing the temperature (or any other field defined on the tmesh), the values in the solid were ignored.

With this change, one may specify which mesh the kernel operates on alongside each vector of fields. If this is omitted, the vmesh is used.

The conj_ht example is not really suited for turbulent averaging, any suggestions where to put an example of the new functionality?

stgeke commented 1 year ago

There is another fix for this issue in next. If all your input fields are of size nrs->fieldOffset it should for CHT as well. Please let me know if we can close this PR.

joneuhauser commented 1 year ago

Oh. What a waste of time then, thanks for the fix! I'll close this MR.