Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add Periodic BCs in FV #29539

Open
GiudGiud opened this issue Dec 13, 2024 Discussed in #29537 · 0 comments
Open

Add Periodic BCs in FV #29539

GiudGiud opened this issue Dec 13, 2024 Discussed in #29537 · 0 comments
Labels
C: Framework T: task An enhancement to the software.

Comments

@GiudGiud
Copy link
Contributor

Reason

Periodic BCs are important for some simulations, often for numerical benchmarks
FE supports them for nodal variables. For finite volume, we do not support them at this time.

Design

Possibly through the functor interface we could:

  • use transformation to go from a functor argument on one boundary to a functor argument on another
  • use a GhostingBoundary relationship manager to ghost the other boundary so it can be used when considering the first one

Impact

Open finite volume to new simulation types

Discussed in #29537

Originally posted by AdrienWehrle December 13, 2024
Hi everyone,

I'm working on an INSFV app and would need to apply periodic boundary conditions to test it on analytical solutions. I can't seem to find any example of the Periodic Action used in FV, and when I'm trying to add it in [BCs] it's simply being disregarded, (which was somehow expected).

The geometry is a simple 2D block so periodicity should be easy to catch.

Here is my input file: https://github.com/AdrienWehrle/diuca/blob/PR_ViscousFlowFVTests/inputs/viscous_diuca/iceslab_fv_2d_SI_nv.i

I'm on moose-libmesh 2024.12.02 and moose-tools 2024.11.13, and the latest version of moose on master.

Thanks a lot in advance for your help!

@GiudGiud GiudGiud added C: Framework T: task An enhancement to the software. labels Dec 13, 2024
GiudGiud added a commit to GiudGiud/moose that referenced this issue Dec 13, 2024
GiudGiud added a commit to GiudGiud/moose that referenced this issue Dec 13, 2024
GiudGiud added a commit to GiudGiud/moose that referenced this issue Dec 24, 2024
GiudGiud added a commit to GiudGiud/moose that referenced this issue Dec 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C: Framework T: task An enhancement to the software.
Projects
None yet
Development

No branches or pull requests

1 participant