Refactor two fluid model#201
Conversation
spossann
left a comment
There was a problem hiding this comment.
This is just the first part of my review.
|
@szekedidavid Can you put the construction of the V0-Derham complex for lifting in the Derham class already? In that way I could start from that, which would be nice. |
…rything that entails
…-hub/struphy into refactor-two-fluid-model
Done! I also removed the boundary condition flags in the propagator. |
Thanks! Will let you know asap. |
|
Hi @szekedidavid - a new notebook is available, see this comment: #212 (comment) |
…rything that entails
caed772 to
61b2955
Compare
…g test case in 2D satisfies BCs, but blows up in the interior.
…e for lifting reconstruction.
|
Hi @spossann , I managed to make lifting work in 1D using the FEECVariable implementation. Also expanded it to 2D and added the full solution (propagator is much, much cleaner now). The 2D lifting test case is not yet working, but it could entirely be a stabilization parameter or time stepping issue (unconstrained tangential velocity DoFs blow up in Hdiv). |
…uphy into refactor-two-fluid-model
|
Hi @szekedidavid - how are you? I managed to merge the branch with the current devel, which featured some re-factoring of how the propagators are stored (now in separate files). I hope that all your major changes are still there, please have a look. Important (!!): I could not get the changes you made in Please contact me in case of uncertainty. |
…n unit test - it works. The Propagator ImplicitDiffusion now allows for this type of bc.
No description provided.