simphony / simphony-openfoam

The implementation of the SimPhoNy OpenFOAM -wrappers.
GNU General Public License v2.0
2 stars 0 forks source link

Solver availability in control and internal wrapper #60

Closed TobiasRasp closed 8 years ago

TobiasRasp commented 8 years ago

By setting CUBAExt.INCOMPRESSIBLE, CUBAExt.LAMINAR_MODEL in the poiseuille example of the fileIO-control wrapper, simpleFoam is used internally to solve the problem. On the other hand, pimpleFoam is applied (by default) for the poiseuille example of the internal wrapper, when the same setting is given.

It would be great, if the same set of OpenFoam solvers (i.e. simpleFoam, pimpleFoam, interFoam) were suported by both wrapper types in the new version (Migration to OF2.4).

santiagomarquezd commented 8 years ago

Hi @TobiasRasp, each solver requires a particular wrapping strategy and the original task was set to work with laminar incompressible and mixture. Our decision for pimpleFoam was due to the problem is transient and several PISO corrections are required in each time-step, which in simpleFoam are not available. I suppose that @khiltunen was focused in the stationary solution for the poiseuille case.

TobiasRasp commented 8 years ago

I agree on the need for a transient solver and pimpleFoam is absolutely fine for me! My point is more about the fact that pimpleFoam is not supported in the IOcontrolwrapper (or vice versa simpleFoam is not supported in the internal wrapper). And it is a bit confusing that the identical input example triggers two different solvers (pimple for internal / simple for control wrapper) when currently switching between both wrapper versions.

khiltunen commented 8 years ago

@TobiasRasp this will be fixed on the migration to OF2.4. Default solver will be pimpleFoam also on IO wrapper.

khiltunen commented 8 years ago

Fixed in v 0.2.0