View Issue Details

IDProjectCategoryView StatusLast Update
0001286OpenFOAM[All Projects] Bugpublic2014-12-18 13:18
ReporterdkxlsAssigned Tohenry 
PrioritynormalSeveritytextReproducibilityalways
Status resolvedResolutionfixed 
PlatformLinux x86_64OSopenSUSEOS Version12.3
Product Version 
Fixed in Version 
Summary0001286: [SprayParcel]: Incorrect tc and ms for new parcels added by the breakup model
DescriptionThe characteristic time scale tc should be 0 for newly introduced droplets to allow for RT breakup for child droplets.

The stripped mass for child droplets should be -GREAT to avoid further stripping of droplets from the child droplets.

The attached patch corrects this to comply with the old dieselSpray implementation (for a further reference see Beale and Reitz (1999)).
Additional InformationBeale, J.C.; Reitz, R.D., "Modeling Spray Atomization with the Kelvin-Helmholtz/Rayleigh-Taylor Hybrid Model," Atomization and Sprays, Vol. 9, pp. 623-650, 1999
Tagsbreakup, Lagrangian, spray

Activities

dkxls

2014-05-10 14:58

reporter  

0001-lagrangian-SprayParcel-Correct-tc-and-ms-for-new-par.patch (788 bytes)
diff --git a/src/lagrangian/spray/parcels/Templates/SprayParcel/SprayParcel.C b/src/lagrangian/spray/parcels/Templates/SprayParcel/SprayParcel.C
index 5a60235..63f4e74 100644
--- a/src/lagrangian/spray/parcels/Templates/SprayParcel/SprayParcel.C
+++ b/src/lagrangian/spray/parcels/Templates/SprayParcel/SprayParcel.C
@@ -307,8 +307,8 @@ void Foam::SprayParcel<ParcelType>::calcBreakup
         child->KHindex() = 1.0;
         child->y() = td.cloud().breakup().y0();
         child->yDot() = td.cloud().breakup().yDot0();
-        child->tc() = -GREAT;
-        child->ms() = 0.0;
+        child->tc() = 0.0;
+        child->ms() = -GREAT;
         child->injector() = this->injector();
         child->tMom() = massChild/(Fcp.Sp() + Fncp.Sp());
         child->user() = 0.0;
--
1.8.1.4

dkxls

2014-12-17 16:18

reporter   ~0003328

This bug is still present in 2.3.1!

henry

2014-12-18 13:18

manager   ~0003337

Thanks for the bug-report and patch
Resolved by commit a55f8a1712523f96d7a97c4d842783a51e6a9ea9

Issue History

Date Modified Username Field Change
2014-05-10 14:58 dkxls New Issue
2014-05-10 14:58 dkxls File Added: 0001-lagrangian-SprayParcel-Correct-tc-and-ms-for-new-par.patch
2014-06-11 16:10 dkxls Tag Attached: breakup
2014-06-11 16:10 dkxls Tag Attached: Lagrangian
2014-06-11 16:10 dkxls Tag Attached: spray
2014-12-17 16:18 dkxls Note Added: 0003328
2014-12-18 13:18 henry Note Added: 0003337
2014-12-18 13:18 henry Status new => resolved
2014-12-18 13:18 henry Resolution open => fixed
2014-12-18 13:18 henry Assigned To => henry