View Issue Details

Jump to NotesJump to History
IDProjectCategoryView StatusDate SubmittedLast Update
0001505OpenFOAMBugpublic2015-02-01 17:492015-02-02 09:38
ReporterJuho Assigned Tohenry  
PrioritynormalSeveritymajorReproducibilityN/A
Status resolvedResolutionfixed 
Summary0001505: [twoPhaseEulerFoam] EoH2 calculation
DescriptionIn function Foam::phasePair::EoH2() the dispersed phase hydraulic diameter is calculated by dividing diameter by a cuberoot of Eotvos number instead of cuberoot of aspect ratio.
Additional InformationCurrent implementation:
        EoH
        (
            dispersed().d()
           /cbrt(Eo())
        );

Corrected:
        EoH
        (
            dispersed().d()
           /cbrt(E())
        );

The corrected EoH2 formulation matches the alternative hardcoded EoH1 implementation if the Wellek aspect ratio model is used.
TagsNo tags attached.

Activities

will

2015-02-02 09:28

manager   ~0003647

Yes, I just dug up the original references, and I can confirm both the bug and the proposed change are accurate.

henry

2015-02-02 09:38

manager   ~0003648

Resolved by commit 1baba9a2d191cbfea73a10eaf0280235a6fee0da

Issue History

Date Modified Username Field Change
2015-02-01 17:49 Juho New Issue
2015-02-02 09:28 will Note Added: 0003647
2015-02-02 09:38 henry Note Added: 0003648
2015-02-02 09:38 henry Status new => resolved
2015-02-02 09:38 henry Resolution open => fixed
2015-02-02 09:38 henry Assigned To => henry