View Issue Details

IDProjectCategoryView StatusLast Update
0003645OpenFOAMBugpublic2021-03-21 10:18
Reporterlhp Assigned Tohenry  
PrioritylowSeverityminorReproducibilityN/A
Status resolvedResolutionfixed 
PlatformLinuxOSUbuntuOS Version20.04.2 LTS
Summary0003645: I believe that there is a typo in both molecular weight definitions, thus neglecting phase2 W()
DescriptionThe aforementioned typo is found under the following file:

$FOAM_SOLVERS/multiphase/compressibleInterFoam/twoPhaseMixtureThermo/twoPhaseMixtureThermo.C

These are the corresponding lines of code, where I believe there should be alpha2()*thermo2_->W() instead:

381 Foam::tmp<Foam::volScalarField> Foam::twoPhaseMixtureThermo::W() const
382 {
383 return alpha1()*thermo1_->W() + alpha2()*thermo1_->W();
384 }
385
386
387 Foam::tmp<Foam::scalarField> Foam::twoPhaseMixtureThermo::W
388 (
389 const label patchi
390 ) const
391 {
392 return
393 alpha1().boundaryField()[patchi]*thermo1_->W(patchi)
394 + alpha2().boundaryField()[patchi]*thermo1_->W(patchi);
395 }

If this is the case for a fix, please have a look at the dev branch as well.
TagsNo tags attached.

Activities

henry

2021-03-21 10:13

manager   ~0011948

Resolved in OpenFOAM-8 by commit 8fc255467d794e0f6e6fcab4113472ae6f782c13
Resolved in OpenFOAM-dev by commit 8ec5eac35e6bc608a69f05e87d34a0ee1370d74c

Issue History

Date Modified Username Field Change
2021-03-21 09:48 lhp New Issue
2021-03-21 10:13 henry Assigned To => henry
2021-03-21 10:13 henry Status new => resolved
2021-03-21 10:13 henry Resolution open => fixed
2021-03-21 10:13 henry Fixed in Version => 8
2021-03-21 10:13 henry Note Added: 0011948