ReynoldsAnalogy CHT model uses incorrect name to look-up fluidThermo

Summary

The ReynoldsAnalogy model uses name to look-up the fluidThermo model.

Steps to reproduce

Define a heatTransferCoeff function with the ReynoldsAnalogy HTC model and use it with a compressible solver without defining the CpInf.

Example:

myCoeff
{
  type            heatTransferCoeff;
  libs            ("libfieldFunctionObjects.so");

  field           T;
  htcModel        ReynoldsAnalogy;
  patches         (my_wall);
  UInf            (1 0 0);

  executeControl  writeTime;
  writeControl    writeTime;
}

What is the current bug behaviour?

An error is displayed:

--> FOAM FATAL ERROR: Unable to set Cp for patch 2

From function virtual Foam::tmp<Foam::Field<double> > Foam::heatTransferCoeffModels::ReynoldsAnalogy::Cp(Foam::label) const
in file heatTransferCoeff/heatTransferCoeffModels/ReynoldsAnalogy/ReynoldsAnalogy.C at line 93.

FOAM exiting

What is the expected correct behavior?

The heatTransferCoeff function should look-up the fluidThermo model and compute the specific heat.

Possible fixes

In the file ReynoldsAnalogy.C at lines 84 and 87 replace fluidThermo::typeName with fluidThermo::dictName.

closed via commit 65342453

Thanks - fixed in master (1912), will get merged into develop before release.

Great, thanks!

While we are at this issue, it would be nice to add this function object to templates in the etc\caseDicts. I do not know if I should open a separate issue for that?

Good enough to just attach it here. If you have a template you would like to see. Would put it in develop though (not a bug, but an enhancement).

Thanks!

reopened

I would make the template like this, other settings should probably be defined explicitly:

/*--------------------------------*- C++ -*----------------------------------*\
  =========                 |
  \\      /  F ield         | OpenFOAM: The Open Source CFD Toolbox
   \\    /   O peration     |
    \\  /    A nd           | Web:      www.OpenFOAM.com
     \\/     M anipulation  |
-------------------------------------------------------------------------------
Description
    Calculates the heat transfer coefficient at wall patches,
    outputting the data as a volScalarField.

\*---------------------------------------------------------------------------*/

type            heatTransferCoeff;
libs            ("libfieldFunctionObjects.so");

field           T;

executeControl  writeTime;
writeControl    writeTime;

// ************************************************************************* //

heatTransferCoeff? or ReynoldsAnalogy? Now not sure which one you meant.

heatTransferCoeff so that it can be used by postProcessing framework.

mentioned in commit d93dd11a

A minimum working example for postProcess -func heatTransferCoeff should involve the following entries, for most of them user input is case dependent:

type            heatTransferCoeff;
libs            (fieldFunctionObjects);
field           <field>;
patches         (<patch1> <patch2> ... <patchN>);
htcModel        <htcModel>;

And there will be further conditionally mandatory entries depending on htcModel (there are three htcModel):

UInf            (20 0 0);
Cp              CpInf;
CpInf           1000;
rho             rhoInf;
rhoInf          1.2;

Might be better to leave it out after the release?

Already in. Seems like a good start, helps @daniel.jasinski, can always refine afterwards.

mentioned in commit 65342453

closed