View Issue Details

IDProjectCategoryView StatusLast Update
0002613OpenFOAMContributionpublic2017-07-13 17:06
ReporterTomF Assigned Tohenry  
PrioritynormalSeverityminorReproducibilityalways
Status resolvedResolutionfixed 
PlatformOpenSUSEOSLeapOS Version42.1
Fixed in Versiondev 
Summary0002613: swirlFlowRateInletVelocity does not give expected result for axi-symmetric case
DescriptionFor an axisymmetric case the swirlinletvelocity gives the swirl at the center of the inlet patch instead of on the axis. This contribution allows the use to select the axis and origin. Also attached is a small demo case
Steps To Reproduce1. Run demo case with swirlInletFlowRateVelocity

2. compile axisSwirlInletFlowRateVelocity

3. Run demo case with axisSwirlInletFlowRateVelocity on the inlet patch
TagsNo tags attached.

Activities

TomF

2017-07-13 14:45

reporter  

henry

2017-07-13 15:12

manager   ~0008389

Coincidentally I have been working on this generalization today and on additional functionality to specify the radial velocity distribution. I will check your version against mine and merge functionality.

henry

2017-07-13 15:15

manager   ~0008390

Your version replaces the current functionality but it would be easy to support both in the same class. I will push my version later today.

TomF

2017-07-13 15:19

reporter   ~0008391

I suppose having it in the same class would indeed be the preferred option.

henry

2017-07-13 16:10

manager   ~0008392

Resolved by commit 371bec2048cd5da74cce97f14007753ee6b80297

Please re-open if there is an issue with this generalized implementation.

TomF

2017-07-13 16:54

reporter   ~0008393

No issue with the functionality, just the suggestion to add the default origin and axis in the description section. i.e.

Property | Description | Required | Default value
phi | flux field name | no | phi
rho | density field name | no | rho
origin | origin of rotation in Cartesian co-ordinates | no | centerpoint of patch
axis | axis of rotation | no | normal of patch
flowRate | flow rate profile | yes |
rpm | rotational speed profile | yes |

henry

2017-07-13 17:01

manager   ~0008394

I will add documentation. Note the default axis is actually -patch normal.

henry

2017-07-13 17:06

manager   ~0008395

Resolved by commit 205ab925e596cbe9ee5a41dad461c2e54f0f7536

Issue History

Date Modified Username Field Change
2017-07-13 14:45 TomF New Issue
2017-07-13 14:45 TomF File Added: axisSwirlFlowRateInletVelocity.tgz
2017-07-13 15:12 henry Note Added: 0008389
2017-07-13 15:15 henry Note Added: 0008390
2017-07-13 15:19 TomF Note Added: 0008391
2017-07-13 16:10 henry Assigned To => henry
2017-07-13 16:10 henry Status new => resolved
2017-07-13 16:10 henry Resolution open => fixed
2017-07-13 16:10 henry Fixed in Version => dev
2017-07-13 16:10 henry Note Added: 0008392
2017-07-13 16:54 TomF Status resolved => feedback
2017-07-13 16:54 TomF Resolution fixed => reopened
2017-07-13 16:54 TomF Note Added: 0008393
2017-07-13 17:01 henry Note Added: 0008394
2017-07-13 17:06 henry Status feedback => resolved
2017-07-13 17:06 henry Resolution reopened => fixed
2017-07-13 17:06 henry Note Added: 0008395