-
Notifications
You must be signed in to change notification settings - Fork 81
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
Implementing hybrid upwinding for total velocity schemes #2394
Conversation
# Conflicts: # src/coreComponents/finiteVolume/FluxApproximationBase.cpp # src/coreComponents/finiteVolume/FluxApproximationBase.hpp # src/coreComponents/physicsSolvers/fluidFlow/IsothermalCompositionalMultiphaseFVMKernels.hpp # src/coreComponents/unitTests/fluidFlowTests/CMakeLists.txt
…ature/jafranc/hybrid-upwinding
…/GEOSX/GEOSX into feature/jafranc/hybrid-upwinding
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanks!
@jafranc will you have time to merge it? |
It is plan for early march as needed for other projects |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## develop #2394 +/- ##
===========================================
- Coverage 53.43% 53.18% -0.25%
===========================================
Files 981 981
Lines 83062 83437 +375
===========================================
- Hits 44382 44380 -2
- Misses 38680 39057 +377 ☔ View full report in Codecov by Sentry. |
…/GEOS-DEV/GEOS into feature/jafranc/hybrid-upwinding
this does not seem to require baselines. |
Following up #2410, this PR aims at implementing simple version of Implicit Hybrid Upwinding (IHU). It implements
IHUPhaseFlux
equivalent toC1PPUPhaseFlux
orPPUPhaseFlux
located inIsothermalCompositionalMultiphaseFVMKernelUtilities.hpp
. It serves as a dispatch value inFaceBasedAssemblyKernelFactory::createAndLaunch()
.some refs on IHU:
"Implicit Hybrid Upwinding for two-phase flow in heterogeneous porous media with buoyancy and capillarity"(Hamon et al,2018)
"Consistent upwinding for sequential fully implicit multiscale compositional simulation" (Moncorge et al,2020)
"Implicit Hybrid Upwinding for two-phase flow in heterogeneous porous media with buoyancy and capillarity"(Hamon et al,2018)
It also introduces
UpwindHelpers
namespace to scope helpers functions computing upwinded fractional flows and potential fluxes that are useful in IHU (and other schemes). Eventually, it introduces an Abstract classUpwindScheme
introducing high level mechanisms to compute upwind direction and fluxes and its Derived classHybridUpwind
to implement specifics to IHU.It could be complemented and extended by adding new upwinding schemes such as Phase Upwind (PU) or weighted Hybrid Upwind (wIHU)
"Smooth implicit hybrid upwinding for compositional multiphase flow in porous media"(Bosma et al, 2022)
"Upstream Differencing for Multiphase Flow in Reservoir Simulation"