View Issue Details

IDProjectCategoryView StatusLast Update
0001447OpenFOAM[All Projects] Bugpublic2014-12-16 08:45
Reporteruser1037Assigned Tohenry 
PrioritynormalSeverityminorReproducibilityalways
Status resolvedResolutionfixed 
PlatformLinuxOSUbuntuOS Version14.04
Product Version 
Fixed in Version 
Summary0001447: Calculation of thermal resistance layers in compressible::turbulentTemperatureCoupledBaffleMixed wrong
DescriptionWhile using the boundary condition "compressible::turbulentTemperatureCoupledBaffleMixed" to couple regions, it is possible to add thin thermal resistance layers using the keywords
thicknessLayers ();
kappaLayers ();
However, the calculation of the combined thermal resistance of more than one layer is wrong.
Used equation: contactRes_ = sum(k_i/l_i)
Correct equation: contactRes_ = 1/sum(l_i/k_i)
Steps To ReproduceRun a case (eg. attached file) with

thicknessLayers (2e-3);
kappaLayers (1e-5);

and compare with
 
thicknessLayers (1e-3 1e-3);
kappaLayers (1e-5 1e-5);
Additional InformationPossible fix:
Change the code in turbulentTemperatureCoupledBaffleMixedFvPatchScalarField.C
from (line 120-127)

forAll (thicknessLayers_, iLayer)
{
    const scalar l = thicknessLayers_[iLayer];
    if (l > 0.0)
    {
        contactRes_ += kappaLayers_[iLayer]/l;
    }
}

to

forAll (thicknessLayers_, iLayer)
{
    const scalar l = thicknessLayers_[iLayer];
    if (l > 0.0)
    {
        contactRes_ += l/kappaLayers_[iLayer];
    }
}
contactRes_ = 1/contactRes_;
TagsNo tags attached.

Activities

dkxls

2014-11-19 16:00

reporter   ~0003291

Last edited: 2014-11-19 16:01

View 2 revisions

Have you checked bug #1440 and the fix in commit c5944c539043440b6c3ac607a359dd6ddddf3184?

user1037

2014-11-20 06:55

 

testCase_kappaLayersBug.tar.gz (14,862 bytes)

user1037

2014-11-20 06:59

  ~0003292

I actually missed that one, thank you.
It is the same error, so the same fix should be applied to the boundary without radiation as well.

henry

2014-12-16 08:44

manager   ~0003320

Thanks for the bug-report.
Resolved by commit 87653bd27dbf2efa62bf2075794c703ac304dc65

Issue History

Date Modified Username Field Change
2014-11-19 14:13 user1037 New Issue
2014-11-19 16:00 dkxls Note Added: 0003291
2014-11-19 16:01 dkxls Note Edited: 0003291 View Revisions
2014-11-20 06:55 user1037 File Added: testCase_kappaLayersBug.tar.gz
2014-11-20 06:59 user1037 Note Added: 0003292
2014-12-16 08:44 henry Note Added: 0003320
2014-12-16 08:44 henry Status new => resolved
2014-12-16 08:44 henry Resolution open => fixed
2014-12-16 08:44 henry Assigned To => henry