|
[Sponsors] |
September 2, 2008, 02:45 |
Dear Developers,
Just encou
|
#1 |
Member
Niklas Wikstrom
Join Date: Mar 2009
Posts: 86
Rep Power: 17 |
Dear Developers,
Just encountered a minor inconsistency: In LESProperties, conveniently all constants are represented by scalars, however PrandtlDelta performs dimensionedScalar lookups. OpenFOAM version 1.5.x (Pulled from git repo) Affected files $FOAM_SRC/turbulenceModels/LES/LESdeltas/PrandtlDelta/PrandtlDelta.C Simple Fix: 65c65 < kappa_(dimensionedScalar(dd.lookup("kappa")).value ()), --- > kappa_(readScalar(dd.lookup("kappa"))), 68,69c68 < dimensionedScalar(dd.subDict(type() + "Coeffs").lookup("Cdelta")) < .value() --- > readScalar(dd.subDict(type() + "Coeffs").lookup("Cdelta")) 83,84c82,83 < kappa_ = dimensionedScalar(d.lookup("kappa")).value(); < Cdelta_ = dimensionedScalar(dd.lookup("Cdelta")).value(); --- > kappa_ = readScalar(d.lookup("kappa")); > Cdelta_ = readScalar(dd.lookup("Cdelta")); Cheers Niklas |
|
September 2, 2008, 05:19 |
Thanks for the bug-report, the
|
#2 |
Senior Member
Join Date: Mar 2009
Posts: 854
Rep Power: 22 |
Thanks for the bug-report, the fix is now in the 1.5.x git repository.
H |
|
August 9, 2010, 08:59 |
|
#3 |
Member
Sylvain Aguinaga
Join Date: Feb 2010
Posts: 41
Rep Power: 16 |
It looks like the proposal of Niklas is not present in 1.7.x version as it still looking for dimensionedScalar Lookups.
When I run pisoFoam it is not finding the kappa constant even if I put it in the dictionary, I don't know yet if it is linked. I will try to recompile the code with Niklas proposal. I'm not sure, I'm a bit lost among all those models and C code... Sincerely Sylvain |
|
August 9, 2010, 09:52 |
|
#4 |
New Member
Andrew Heather
Join Date: Mar 2009
Posts: 16
Rep Power: 17 |
Thanks - I've re-added the code to 1.7.x - should be there the next time you 'git pull'
Andy |
|
|
|
Similar Threads | ||||
Thread | Thread Starter | Forum | Replies | Last Post |
OF15 functionObjects do not read LESProperties for incompressible flows | aunola | OpenFOAM Bugs | 1 | September 28, 2008 17:18 |