What's missing
AbstractBC is exactly Periodic, Dirichlet, Neumann, Interface. _require_face_bc (src/boundaries.jl:82) errors loudly for anything else, so a Robin condition is a genuine extension rather than a config option — it needs _bc_sign, _offset_ghost! and a _require_face_bc method, plus the matching fold_bc! row and the operator_diagonal correction.
Why (downstream)
Robin is the native boundary condition for trans-wall exchange in the microvascular model, in two places:
- Perivascular wall coupling. Flux across the capillary wall is
g·(C_blood − C_tissue) — a conductance times a concentration difference. That is a Robin condition on the tissue side, currently hand-assembled as a separate coupling operator with its own implicit/explicit split.
- Proton leak regularization. The transported proton field
H is no-flux on every boundary, which makes its Laplacian pure-Neumann and therefore singular. A small Robin leak gH is what regularizes it. Right now that regularization is a bespoke term rather than a boundary condition, so it is invisible to operator_diagonal and to multigrid.
Both are the same mathematical object, and both would be one BC type if MFO had it.
Sketch
Robin(a, b, c) # a·u + b·∂u/∂n = c ; Dirichlet is b=0, Neumann is a=0
Worth deciding up front whether a, b, c may be per-face Fields or must be scalars. The downstream conductance g is genuinely per-face (it varies with local vessel geometry), so scalar-only would leave the main use case unserved — but per-face coefficients interact with the affine boundary_rhs lift, so it's a real design call, not a detail.
Payoff beyond the downstream package
operator_diagonal gaining a Robin case is what lets MultigridPreconditioner precondition wall-coupled systems. That matters: the downstream steady solve currently reaches for ILU(0) on an assembled sparse W precisely because unpreconditioned matrix-free GMRES spent ~750 iterations per linear solve. The system is diffusion-dominated — ideal MG territory — so a Robin-aware diagonal is the step that makes MFO's native preconditioner applicable and lets the sparse-matrix detour go away.
🤖 Beep — Claude wrote this one; Kyle is not yet talking to himself in his own issue tracker.
What's missing
AbstractBCis exactlyPeriodic,Dirichlet,Neumann,Interface._require_face_bc(src/boundaries.jl:82) errors loudly for anything else, so a Robin condition is a genuine extension rather than a config option — it needs_bc_sign,_offset_ghost!and a_require_face_bcmethod, plus the matchingfold_bc!row and theoperator_diagonalcorrection.Why (downstream)
Robin is the native boundary condition for trans-wall exchange in the microvascular model, in two places:
g·(C_blood − C_tissue)— a conductance times a concentration difference. That is a Robin condition on the tissue side, currently hand-assembled as a separate coupling operator with its own implicit/explicit split.His no-flux on every boundary, which makes its Laplacian pure-Neumann and therefore singular. A small Robin leakgHis what regularizes it. Right now that regularization is a bespoke term rather than a boundary condition, so it is invisible tooperator_diagonaland to multigrid.Both are the same mathematical object, and both would be one BC type if MFO had it.
Sketch
Worth deciding up front whether
a,b,cmay be per-faceFields or must be scalars. The downstream conductancegis genuinely per-face (it varies with local vessel geometry), so scalar-only would leave the main use case unserved — but per-face coefficients interact with the affineboundary_rhslift, so it's a real design call, not a detail.Payoff beyond the downstream package
operator_diagonalgaining a Robin case is what letsMultigridPreconditionerprecondition wall-coupled systems. That matters: the downstream steady solve currently reaches for ILU(0) on an assembled sparseWprecisely because unpreconditioned matrix-free GMRES spent ~750 iterations per linear solve. The system is diffusion-dominated — ideal MG territory — so a Robin-aware diagonal is the step that makes MFO's native preconditioner applicable and lets the sparse-matrix detour go away.🤖 Beep — Claude wrote this one; Kyle is not yet talking to himself in his own issue tracker.