Adding the Energy Equation to interFoam (OF 2.4.0)
 

Dear Foamers,

first of all, this is my first thread, me being new to OF and the forum as well. So I am thankful for any advice regarding anything related to CFD in general and OF and this forum in particular.

Adding the Energy Equation to interFaom makes it necessary to modify the used library (A) and secondly modify the solver itself (B). I am having problems compiling said application, hence I have written this thread.
I am going to describe my steps in detail in this thread. The procedure is inspired by both the documents from Damiano Natali ("interTempFoam") and Qingming Liu ("myinterFoamDiabatic"). The links are given beneath.

http://www.wolfdynamics.com/images/c...erTempFoam.pdf

http://www.tfd.chalmers.se/~hani/kur...gLIU-final.pdf

A) Modifcation of the library

1)
Copy the original folder transportModels to the user's directory

cd $WM_PROJECT_USER_DIR/src/
cp -rp $FOAM_SRC/transportModels .

2)
Rename the folders and files which are being modified (namely the libraries incompressibleTwoPhaseMixture and immiscibleIncompressibleTwoPhaseMixture).

cd transportModels/incompressible
mv incompressibleTwoPhaseMixture myIncompressibleTwoPhaseMixture
cd myIncompressibleTwoPhaseMixture
mv incompressibleTwoPhaseMixture.C myIncompressibleTwoPhaseMixture.C
mv incompressibleTwoPhaseMixture.H myIncompressibleTwoPhaseMixture.H

cd ../..
mv immiscibleIncompressibleTwoPhaseMixture myImmiscibleIncompressibleTwoPhaseMixture
cd myImmiscibleIncompressibleTwoPhaseMixture
mv immiscibleIncompressibleTwoPhaseMixture.C myImmiscibleIncompressibleTwoPhaseMixture.C
mv immiscibleIncompressibleTwoPhaseMixture.H myImmiscibleIncompressibleTwoPhaseMixture.H

3)
Change the respective file files corresponding to the the naming and path.
in the folder incompressible/Make for myIncompressibleTwoPhaseMixture:
in the folder myImmiscibleIncompressibleTwoPhaseMixture/Make for myImmiscibleIncompressibleTwoPhaseMixture:
4) Make sure the newly named files are being included with their new name in the respective files:
in myIncompressibleTwoPhaseMixture.C:
in myImmiscibleIncompressibleTwoPhaseMixture.C
in myImmiscibleIncompressibleTwoPhaseMixture.H
5)
In the file myImmiscibleIncompressibleTwoPhaseMixture/Make/options make the adaption as highlighted, so the connection to the new library is made.
6)
Add the new parameters rho and Pr in myIncompressibleTwoPhaseMixture.H.
and
also for kappaf
7)
In myIncompressibleTwoPhaseMixture.C add the operations for the new parameters.
and
Also add the lines for the calculation of kappaf().
8)
Compile both libraries from the respective folder. Start with myIncompressibleTwoPhaseMixture, because myImmiscibleIncompressibleTwoPhaseMixture depends on it already being compiled beforehand.

cd incompressible
wclean
wmake libso
cd ..
cd myImmiscibleIncompressibleTwoPhaseMixture
wclean
wmake libso


Compiling works at this stage, which is good for the moment but doesn't necessarily mean to me that I did everything right.



B) Modification of the solver interFoam to interTempFoam

1)
Enter the destination of the new solver and copy the base solver.

cd $WM_PROJECT_USER_DIR/applications/solvers/multiphase
cp -rp $FOAM_APP/solvers/multiphase/interFoam .

2)
Rename the folder and the .C file.

mv interFoam interTempFoam
cd interTempFoam
mv interFoam.C interTempFoam.C

3)
Adapt the file Make/files to the changed naming and path.
4)
In interTempFoam.C adapt the name of the following library being included.
5)
Add the following lines in the file createFields.H.
and
and
6)
In alphaEqn.H add:
and in alphaEqnSubCycle.H add:
7)
Create a file named Teqn.H in the interTempFoam folder containing the Energy Equation:
8)
Add the command for solving the E-Eqn in interTempFoam.C. The specific location of the added line may be object of discussion. Here is my suggestion:
9)
Adapt the file Make/options to the names and locations of the beforehand modified libraries. The following code is probably not correct and thus object of my questions.
9)
Compile the solver from within its directory interTempFoam.

wclean
wmake

<------------------------------------------------------------------------------------------------------------------------->
When I do everything as described I get the following output log:
Let us prioritize the error over the warning.
Do you have any idea how it comes to that error and how to solve it?

I would very much appreciate comments from your side.

Best Regards
Perry

interTempFoam in OF 2.3.1
 

Hi,
I'm working on the exact same problem. I've used the wolfdynamics page to make nearly same modifications as you to incorporate temperature into OpenFOAM (2.3.1), and I am getting a similar error in the end. The only differences in my procedure are:

Using "mixture" in your step 7: This is used to refer to the other quantities defined in myIncompressibleTwoPhaseMixture, so that's why I've used it. Anything else gives me the error you got.
Then in your step 8: I've added the include call immediately after the Ueqn.H one.
Then in your step 9: I've essentially tried to add any libraries that might help.
This then gives me the following error while compiling:
This is actually an error that has turned up a few times when I search the forums, but the solutions that have worked for people (compiling the libraries before the solver, referencing the modified libraries rather than those in interFoam) are not apparently the problem in this case.

Does anyone have any suggestions?

Fantastic post Perry, you are a role model for this forum. Anyway, you did some strange things:
1. Your new library has the same name as the old one
2. You include both the new and old library in the Make files, although the old one is completely redundant.

Hi Hannah,

thank you for your reply. I implemented the first line of your TEqn.H file:
For the time being I left the rest as it is, because it succesfully compiled (when one ignores the warning about maxDeltaT).

I just adjusted the given damBreak tutorial case (with its very coarse mesh). It completed the calculation, but with temperature values beneath zero. In consequence I will try a refined mesh and I will probably have to look into fvSchemses and fvSolution in more detail.

Greetings
Perry

Hello Anton,

thanks for your feedback and the compliment.
I am guessing with "the old one [library]" you are referring to the directory "incompressible", the path of which is specified in:
If YES: "incompressible" is not the name of any library, but rather the folder in which the library "incompressibleTwoPhaseMixture" or in my case "myIncompressibleTwoPhaseMixture" is situated in.
If NO: What do you mean?

I think I just gave the answer to that question, too.
Or didn't I?

Greetings
Perry

Hi Perry,

I'm glad that little change worked. I've modified mine to exactly match yours (with the mixture.kappaf) and it now compiles, too! Thanks for your very detailed description of all of your interFoam modifications. Now to see if it actually works...

Hannah

It will be very difficult to you to obtain solution with conserved energy. Because when you are using interFoam, all properties are advected with volumetric flux. So, you must rewrite equation for temperature for each phase, and then, by summing for all phases, you can get equation for temperature of mixture:

Unfortunately, I get a segmentation fault when I run my newly compiled interFoam with temperature for a modified damBreak case with temperature. How do I interpret the error message to identify what might be wrong?

To modify damBreak I have added a T file to the 0 folder, cp and Pr to both phases in the transportProperties,

to fvSchemes,

to fvSolution, and initial conditions to setFields.

I'd appreciate help identifying where the error might be so that I might know what to do to fix it.

mkraposhin, that's an interesting thought. Once this actually runs, then I'll be interested in getting the best solution possible.

Hannah

from this lines it is seen that you are still using old model
immiscibleIncompressibleTwoPhaseMixture

can you post your createFields.H file?

Also, i think that it would be better to nest child class from mixture model, rather then creating new. Because at next lines OF creates turbulence model, that uses mixture model

Here is the createFields.H.

As in Perry's steps and http://www.wolfdynamics.com/images/c...erTempFoam.pdf, I changed the names of the libraries, but not the names of the definitions within them. For instance, myIncompressibleTwoPhaseMixture.C begins like:

Therefore, when I'm calling "immiscibleIncompressibleTwoPhaseMixture mixture(U, phi);" that's been defined within the myImmiscibleIncompressibleTwoPhaseMixture library and so should reference the new versions. All the other libraries are left unchanged, so the turbulence model used is the old one stored in $FOAM_SRC. That said, is there a place in the Make/options modifications described in Perry's first post that would link both old and new libraries and create confusion? That could be a source of the problem...

Sorry for the confusion, I just realized I didn't read it properly and indeed you don't have the original immiscibleIncompressibleTwoPhaseMixture in Make/*.

As i expected, you left initialization for old data type of mixture.
So, you need to change immiscibleIncompressibleTwoPhaseMixture to myImmiscibleIncompressibleTwoPhaseMixture

But when you do that, i think, that solver will fail to compile, because turbulence model class depends on mixture class.

Later i can try to explain how to derive your own class from class immiscibleIncompressibleTwoPhaseMixture

O.k., i'm sorry,

i didn't read untill the end of your post, but nevertheless, turbulence model needs original data type, but not modified, that's why it fails to run

and of course, you must avoid using same data type names for different classes

Hi, i spend ~ 20 minutes and made child class myImmiscibleIncompressibleTwoPhaseMixture, which can do all work without substituting user type instead of original OF

Step 1. Create your new class myImmiscibleIncompressibleTwoPhaseMixture as a sub class of immiscibleIncompressibleTwoPhaseMixture, see attachment files .H and .C.
Please note that for compatibility with OF you need to change all names, for example:
Otherwise, you may get in conflict with OF

Step 2. Adjust files Make/files and Make/options to include all neccesary files from OF
Make/files
Make/options
Step 3. Compile with wmake libso

Step 4. Initialize new model in new solver:
- add -I../myImmiscibleIncompressibleTwoPhaseMixture \ line to Make/options EXE_INC section of your solver
- add consequent lines to -L section of Make/options file of your solver
- add #include "myImmiscibleIncompressibleTwoPhaseMixture.H" to solver instead of OF original solver
- add initialization of your myImmiscibleIncompressibleTwoPhaseMixture class instead of OF original class in createFields.H
- use method mixture.lambdaEff() to get mixture heat conduction coefficient
- solve equation for temperature, formulated in volumetric fluxes:

Thanks so much for your efforts. I'll go through and change all the internal names as well. Since all the new properties are defined in myIncompressibleTwoPhaseMixture (the immiscible one just calls them), that will need to be adjusted, too.

Edit: I see you've suggested moving all of the definitions into myImmiscibleIncompressibleTwoPhaseMixture. I'll incorporate that.

This doesn't help explain, though, why this isn't needed for Perry and everyone else using these (http://www.wolfdynamics.com/images/c...erTempFoam.pdf, http://www.cfd-online.com/Forums/ope...interfoam.html, etc) instructions.

Hannah

Well those changes just produced a lot of compiling errors, but continuing to mess around with my original Make/options and #include's produced a version that compiled and ran a test case successfully. I'll look into making mkraposhin's suggested modifications to TEqn.

In the meantime, I'm attaching all of the files in case they might help someone else (using OF 2.3.1).

Hannah

I will try to compile your code and report what is wrong later.
Regarding previous version of library and solver - i made it for OF 2.4.0, not for 2.3.1

Hello Matvey,

first of all, thanks for taking part in the thread.

Does it make such a difference? Because the coefficients in my proposed TEqn are being calculated for the mixture beforehand:
in alphaEqn.H:
and in alphaEqnSubCycle.H:
To your proposed TEqn:
I see that you divided the equation by (rho*cp)
viewing the terms separately:
That is straigt forward

Here I can see, that you split the single laplacian term in two laplacian terms with each of them having the fitting alpha and parameters in it.

Here I see the basic term for the convective temperature transport plus a new term.
What does this new term do? I do not know the operation in there (SuSp(-fvc ...).

Best Regards
Perry

@ Hannah:

Hi, i looked at your code and i understand that you mixed from all versions :)

So, i used my approach to create a new class for mixture and mass flux approach for energy equation. It works for OF 2.3.1. You can find source code in the attachment.

To make solver, go into folder interThermalFoam/myImmiscibleIncompressibleTwoPhaseMixture and run
wmake libso

then go one folder up and run
wmake