-
Notifications
You must be signed in to change notification settings - Fork 96
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
Eulerian multiphase #160
Comments
Yup, thanks. Knock yourself out! I think the later versions of OF Foundation have consolidated around reactingMultiphaseEulerFoam's I/O format so might be worth looking at that as a base. But I'm not sure so do go ahead as you see fit. |
see #163 |
@oliveroxtoby If one considers Perhaps we implement |
Actually, @oliveroxtoby it looks like a lot of these separate solvers have been amalgamated into |
Yes, I definitely wouldn't actually add any reacting flow at this stage. multiphaseEulerFoam is complex enough as it is. Sorry, my comment was a bit misleading, what I meant was that the new consolidated multiphaseEulerFoam in Foundation 11 is, I think, based around the case format of the reactingMultiphaseEulerFoam in earlier Foundation versions and in the OpenCFD versions. I wasn't intending to suggest you actually support reactions. |
Hi @oliveroxtoby ok ... i guess i have implemented the basics of simple reactions in |
A further comment here @oliveroxtoby , I agree with the above actually. The reactions part is going to need a lot of additional stuff to allow for the configurations and reaction definitions, unless we just used some default stuff. On the basic Eulerian stuff though, did you have any thoughts about how configuration for the dispersed phase and interaction could be done? There are again a lot of options. To me, it seems the best option is to add a There is too much required to try to jam this into the existing |
Yes, I don't think the right place for this is 'Physics Selection'. I would suggest putting phase-specific settings (e.g. diameter model) into the material properties specification for each phase. I would also put inter-phase properties under the material properties, but under an additional selection of the 'other phase'. (That would also be the home for surface tension for VOF - it can't be specified at all currently.) |
Ok. Nice idea, that works well.
Ok, I'll see what I can figure out ... so for multiphase, the user specifies at least x1 additional Material, which then populates interphase properties in the bottom of the current Material Properties object? Something like that? For specifying the |
@oliveroxtoby I have been doing some work on E-E multiphase lately and was thinking of putting a PR together for something like this, for the workbench? There are quite a few different solvers to choose from, but we have the VoF stuff already set up.
Perhaps it might be a nice addition ...
Any thoughts and or advice / preferences?
The text was updated successfully, but these errors were encountered: