View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
0001794 | OpenFOAM | Bug | public | 2015-07-22 21:34 | 2015-12-03 13:53 |
Reporter | Assigned To | henry | |||
Priority | normal | Severity | minor | Reproducibility | have not tried |
Status | resolved | Resolution | fixed | ||
Platform | GNU/Linux | OS | OpenSuSE | OS Version | 13.1 |
Summary | 0001794: wenyu drag model in twoPhaseEulerFoam | ||||
Description | wenyu drag model in theory is as follows: K_gs=3/4*Cd*alpha_g*alpha_s*rho_g/diam_s*(U_g-U_s)*pow(alpha_g,2.65); Cd=24/(Re_s)*[1+0.15*pow(Re_s,0.687)] if Re_s<1000; Cd=0.44 if Re_s>1000; Re_s=alpha_g*rho_g*(U_g-U_s)*diam_s/mu_g; In dragModel.C source code(line 108-114), K_gs is defined as: 0.75 *CdRe() *max(pair_.dispersed(), residualAlpha_) *swarmCorrection_->Cs() *pair_.continuous().rho() *pair_.continuous().nu() /sqr(pair_.dispersed().d()); in Wenyu.C (line70-77), the return value is volScalarField CdsRe ( neg(Re - 1000)*24.0*(1.0 + 0.15*pow(Re, 0.687)) + pos(Re - 1000)*0.44*max(Re, residualRe_) ); return CdsRe *pow(alpha2, -2.65) *max(pair_.continuous(), residualAlpha_); But in source code phasePair.C(line 131), Re is defined as Re = magUr()*dispersed().d()/continuous().nu(), not including alpha_g term, so Re_s = alpha_g*Re; and I think that in line70-71, the term Re should be replaced by alpha_g*Re. It is my suggestion, would you like to give me some response? | ||||
Tags | No tags attached. | ||||
|
Looks plausible. That's how CFX does it http://www.arc.vt.edu/ansys_help/cfx_thry/i1304903.html . Ditto this http://www.scansims.org/sims2008/07.pdf . I've tried the change on a couple of fluidised bed cases and, frankly, I see no difference in the results. I think cases like this, which use a mixture of Wen-Yu and Ergun models, tend to be quite Ergun dominated. Do you have a case where the change has a noticeable (ideally beneficial) effect on the results? |
2015-07-30 19:32
|
|
|
Thank you for your reply.I also found that it could not recognize the influence on the fluidised bed simulation results. Wen-Yu model is usually used to calculate dilute phase flow, such as dilute phase pneumatic conveying. The difference between expression could be seen from the attached file Wen-Yu. |
|
The document shows the difference in the functions, sure, but it would be more encouraging to a difference in an actual simulation. Despite that, I'd probably make this change, even without evidence of an improvement of a simulation. It's not likely to affect anything that we have tested, and it does look more correct on paper. |
|
I have corrected the formulation on OpenFOAM-dev: commit a91cf40aff0b595faf067c7d33bf7373e019b284 Author: Henry Weller <http://cfd.direct> Date: Tue Aug 4 13:29:11 2015 +0100 twoPhaseEulerFoam/interfacialModels/dragModels/WenYu: Corrected Re -> Re*alpha_g Resolves bug-report http://www.openfoam.org/mantisbt/view.php?id=1794 Let me know if you are satisfied that this corresponds to the published form. Note the -3.65 coefficient which handles the CdsRes -> CdsRe |
|
I assume that the update in OpenFOAM-dev is correct, if not please re-open this bug-report. |
|
I am sorry to response so late. I just wonder the function of the term "*max(pair_.continuous(), residualAlpha_)". It is for the numerical requirement or for other purpose? Would you like to give me your suggestions? Thank you a lot! |
|
Yes the residualAlpha_ is needed for numerical stability. |
Date Modified | Username | Field | Change |
---|---|---|---|
2015-07-22 21:34 |
|
New Issue | |
2015-07-30 17:08 | will | Note Added: 0005147 | |
2015-07-30 17:09 | will | Note Edited: 0005147 | |
2015-07-30 17:09 | will | Note Edited: 0005147 | |
2015-07-30 19:32 |
|
File Added: Wen-Yu.docx | |
2015-07-30 19:35 |
|
Note Added: 0005148 | |
2015-07-31 09:52 | will | Note Added: 0005149 | |
2015-08-04 13:31 | henry | Note Added: 0005191 | |
2015-08-18 10:59 | henry | Note Added: 0005258 | |
2015-08-18 10:59 | henry | Status | new => resolved |
2015-08-18 10:59 | henry | Resolution | open => fixed |
2015-08-18 10:59 | henry | Assigned To | => henry |
2015-08-26 19:02 |
|
Note Added: 0005302 | |
2015-08-26 19:02 |
|
Status | resolved => feedback |
2015-08-26 19:02 |
|
Resolution | fixed => reopened |
2015-08-26 19:45 | henry | Note Added: 0005304 | |
2015-12-03 13:52 | henry | Status | feedback => resolved |
2015-12-03 13:52 | henry | Resolution | reopened => fixed |