View Issue Details

IDProjectCategoryView StatusLast Update
0000114OpenFOAMBugpublic2010-12-21 13:26
ReporterJuho Assigned Touser4 
PrioritylowSeveritytextReproducibilityN/A
Status resolvedResolutionfixed 
Summary0000114: turbulentTemperatureCoupledBaffleMixed: Inconsistent class description and implemention
DescriptionThe class description for the class

compressible::turbulentTemperatureCoupledBaffleMixedFvPatchScalarField

The decription in the .H file:

"Description
    Mixed boundary condition for temperature, to be used for heat-transfer
    on back-to-back baffles.

    If my temperature is T1, neighbour is T2:

    T1 > T2: my side becomes fixedValue T2 bc, other side becomes fixedGradient."

appears to describe the option 1. outlined in the .C file:

   "// 1. specify above temperature on one side (preferentially the high side)
    // and above gradient on the other. So this will switch between pure
    // fixedvalue and pure fixedgradient"
    
However, the implementation follows the option 2.:

   "// 2. specify gradient and temperature such that the equations are the
    // same on both sides. This leads to the choice of
    // - refGradient = zero gradient
    // - refValue = neighbour value
    // - mixFraction = nbrKDelta / (nbrKDelta + myKDelta())

    this->refValue() = nbrIntFld;
    this->refGrad() = 0.0;
    this->valueFraction() = nbrKDelta / (nbrKDelta + myKDelta());
    mixedFvPatchScalarField::updateCoeffs();"
TagsNo tags attached.

Activities

user4

2010-12-21 13:26

  ~0000194

Thanks for spotting it and reporting. Fixed in 4bec947c566b4c0e805b8e94e25926abd52b5ac2.

Issue History

Date Modified Username Field Change
2010-12-20 13:36 Juho New Issue
2010-12-21 13:26 user4 Note Added: 0000194
2010-12-21 13:26 user4 Status new => resolved
2010-12-21 13:26 user4 Fixed in Version => 1.7.x
2010-12-21 13:26 user4 Resolution open => fixed
2010-12-21 13:26 user4 Assigned To => user4