Skip to content
Snippets Groups Projects

Fix smagorinsky

Merged Markus Holzer requested to merge holzer/lbmpy:FixSmagorinsky into master

The Smagorinsky model was adapted in !93 (merged). This change should ensure that a smagorinsky model can also be added when omega is not a symbol but a number. However, due to a small issue, only collision rules with omega being a number can be generated at the moment. With this MR, this issue is fixed again.

Furthermore, some wrong type hints in the LBMConfig class have been corrected

Merge request reports

Pipeline #34394 passed

Pipeline passed for ae99418e on holzer:FixSmagorinsky

Merged by Michael KuronMichael Kuron 3 years ago (Sep 26, 2021 1:20pm UTC)

Loading

Pipeline #34395 passed

Pipeline passed for 2bfdfe0f on master

Test coverage 90.25% from 0 jobs

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
Loading