View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0000528 | OpenFOAM | Bug | public | 2012-05-03 17:20 | 2012-05-15 16:27 |
| Reporter | Assigned To | ||||
| Priority | normal | Severity | minor | Reproducibility | always |
| Status | resolved | Resolution | fixed | ||
| Platform | Linux | OS | OpenSuse | OS Version | 11.3 |
| Summary | 0000528: wrong radial velocity in cylindricalInletVelocityFvPatchVectorField | ||||
| Description | in line 140 operator==(tangVel + axis_*axialVelocity_ + radialVelocity_*d); the d vector is not normalized so the actual radial velocity imposed by the BC is multiplied by the physical radius. e second bug is the use of axis_ instead of hatAxis which has the same direction but magnitude equal to one by construction. I believe the code should be something like: operator==(tangVel + hatAxis*axialVelocity_ + radialVelocity_*d/mag(d)); | ||||
| Tags | No tags attached. | ||||
|
|
Thanks for the report. The bug was fixed in commit: 15378e2f165c8389f3ddebaa9482940a607f6fbb |
|
|
Actually there is a second bug related to the use of tmp for the vectorField d. With current implementation when updating operator== the code give the following error: temporary deallocated From function const T& Foam::tmp<T>::operator()() const Simply changing line 135 from tmp<vectorField> d = r - (hatAxis & r)*hatAxis; to vectorField d = r - (hatAxis & r)*hatAxis; will solve the issue. |
|
|
Fixed in cae1e2db065268a6d39d0e15e29a860b53705d00. Thanks for reporting! |
| Date Modified | Username | Field | Change |
|---|---|---|---|
| 2012-05-03 17:20 |
|
New Issue | |
| 2012-05-03 17:50 |
|
Assigned To | => user21 |
| 2012-05-03 17:50 |
|
Status | new => assigned |
| 2012-05-04 11:05 |
|
Note Added: 0001305 | |
| 2012-05-04 11:05 |
|
Status | assigned => resolved |
| 2012-05-04 11:05 |
|
Resolution | open => fixed |
| 2012-05-04 11:45 |
|
Note Added: 0001306 | |
| 2012-05-04 11:45 |
|
Status | resolved => feedback |
| 2012-05-04 11:45 |
|
Resolution | fixed => reopened |
| 2012-05-15 16:27 |
|
Note Added: 0001318 | |
| 2012-05-15 16:27 |
|
Status | feedback => resolved |
| 2012-05-15 16:27 |
|
Fixed in Version | => 2.1.x |
| 2012-05-15 16:27 |
|
Resolution | reopened => fixed |
| 2012-05-15 16:27 |
|
Assigned To | user21 => user4 |
