View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
0003471 | OpenFOAM | Bug | public | 2020-03-26 14:47 | 2020-11-21 19:55 |
Reporter | FoamerSv | Assigned To | henry | ||
Priority | normal | Severity | minor | Reproducibility | have not tried |
Status | closed | Resolution | suspended | ||
Platform | GNU/Linux | OS | Ubuntu | OS Version | 14.04 |
Summary | 0003471: Bugs to calculate the thickness of liquid sheet in LISAAtomization | ||||
Description | The codes in LISAAtomization.C#L148 hSheet = volFlowRate/(constant::mathematical::pi*delta*Urel); It is obviously to demonstrate the calculation of thickness of liquid sheet, hSheet. volFlowRate is the volume rate [m3/s]. delta is the radial distance from the center line (injector direction) to the mid-line of the sheet (umbrella rib). Urel is the velocity of parcel. Should not it be calculated by the following sentence? hSheet = volFlowRate/(constant::mathematical::twoPi*delta*Urel); | ||||
Tags | No tags attached. | ||||
|
|
|
I agree with your analysis, but to be confident enough to release the proposed change I need some way of testing it, and some evidence to suggest that it is beneficial. We don't have a tutorial that uses this functionality and there isn't even a usage description in the header. Given that, and with reports stating that the implementation is incorrect, I would be more inclined to remove this model than to patch it. Do you have a case which uses this model, and which demonstrates the need for your proposed change? |
|
Pending test case. |
Date Modified | Username | Field | Change |
---|---|---|---|
2020-03-26 14:47 | FoamerSv | New Issue | |
2020-03-26 14:47 | FoamerSv | File Added: 2020-03-26_182131.png | |
2020-04-16 12:30 | will | Note Added: 0011293 | |
2020-11-21 19:55 | henry | Assigned To | => henry |
2020-11-21 19:55 | henry | Status | new => closed |
2020-11-21 19:55 | henry | Resolution | open => suspended |
2020-11-21 19:55 | henry | Note Added: 0011708 |