Skip to content
Snippets Groups Projects

ENH: specularRadiation: add axisymmetric fvDOM boundary condition

Merged Kutalmış Berçin requested to merge feature-axisymmetric-fvdom-bc into develop
All threads resolved!

Problem

The fvDOM radiation model of OpenFOAM is not applicable to axisymmetric geometries, resulting in:

  • Increased computational costs,
  • Complicated mesh generation,
  • Reduced numerical stability.

Solution

Implement and verify the axisymmetric fvDOM boundary condition elaborated in:

  • Kumar, P., & Eswaran, V. (2013). A methodology to solve 2D and axisymmetric radiative transfer problems using a general 3D solver. Journal of heat transfer, 135(12).

Verification

Based on:

  • Cai, J., & Modest, M. F. (2016). Specular reflective boundary conditions for Discrete Ordinate Methods in Periodic or Symmetric Geometries. In Journal of Physics: Conference Series (Vol. 676, No. 1, p. 012002). IOP Publishing.

Risks

  • No change in existing input/output.
  • The predictions of the condition for the radial heat flux distribution on walls need to be improved.

Metadata

  • Related: #855 (closed)
  • linux64ClangDPInt32Opt (clang13)
  • linux64GccDPInt32Opt
  • linux64GccSPDPInt64Debug
  • Alltest: No new error/No change in existing output
  • Test cases: MR610-02Jun2023-test.zip

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Mattijs Janssens
  • added 1 commit

    • 21103e6e - ENH: specularRadiation: add axisymmetric fvDOM boundary condition

    Compare with previous version

  • Kutalmış Berçin resolved all threads

    resolved all threads

  • added 1 commit

    • 4fca4519 - ENH: specularRadiation: add axisymmetric fvDOM boundary condition

    Compare with previous version

  • Kutalmış Berçin marked this merge request as ready

    marked this merge request as ready

  • Andrew Heather approved this merge request

    approved this merge request

  • Andrew Heather added 40 commits

    added 40 commits

    Compare with previous version

  • Andrew Heather mentioned in commit 4c86664a

    mentioned in commit 4c86664a

  • merged

  • Please register or sign in to reply
    Loading