View Issue Details

Jump to NotesJump to History
IDProjectCategoryView StatusDate SubmittedLast Update
0003991OpenFOAMFeaturepublic2023-06-23 12:312023-06-23 13:17
Reporterdemichie Assigned Tohenry  
PrioritylowSeveritytextReproducibilityN/A
Status closedResolutionno change required 
PlatformUnixOSUbuntuOS Version22.04
Product Versiondev 
Summary0003991: variable name and description
DescriptionIn the module multiphaseEuler, in cellPressureCorrector (from Line 139), the term phigFs is computed, and it is written that it accounts for buoyancy and force fluxes.

        // Combined buoyancy and force fluxes
        PtrList<surfaceScalarField> phigFs(phases.size());

I don't thin the contribution of foce fluxes is in this term anymore, as it was in previous version of OpenFOAM. Now the contribution is added to phiHbyAs at line 207.
If I am correct, maybe it is worth to change the comment at line 139, and to change the name of the variable phigFs.
TagsNo tags attached.

Activities

henry

2023-06-23 13:17

manager   ~0013061

phigFs includes the surface tension force:

                phigFs.set
                (
                    phasei,
                    (
                        (
                           ghSnGradRho
                         - (fvc::interpolate(phase.rho() - rho))
                          *(buoyancy.g & mesh.Sf())
                         - fluid.surfaceTension(phase)*mesh.magSf()
                        )
                    ).ptr()
                );

Issue History

Date Modified Username Field Change
2023-06-23 12:31 demichie New Issue
2023-06-23 13:17 henry Note Added: 0013061
2023-06-23 13:17 henry Assigned To => henry
2023-06-23 13:17 henry Status new => closed
2023-06-23 13:17 henry Resolution open => no change required