Skip to content
Snippets Groups Projects

BUG: add switch for nu:DphitEff in kEpsilonPhitF (fixes #1560)

Merged Kutalmış Berçin requested to merge bug-1560-kEpsilonPhitF into master

Including nu in DphitEff even though it is not present in (LUU:Eq. 17) provided higher level of resemblance to benchmarks for the tests considered, particularly for the peak skin friction (yet, pressure-related predictions were unaffected). Users can switch off nu in DphitEff by using includeNu entry in kEpsilonPhitFCoeffs in order tofollow the reference paper thereat. includeNu is left true by default. See GitLab issue #1560 (closed),

  • removes redundant phit() and f() access funcs,
  • replaces dimensionedScalar with dimScalar to gain space.

LUU: Laurence, D. R., Uribe, J. C., & Utyuzhnikov, S. V. (2005).

Merge request reports

Checking approval status

Merged by Andrew HeatherAndrew Heather 5 years ago (Jan 21, 2020 12:28pm UTC)

Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply