View Issue Details

IDProjectCategoryView StatusLast Update
0003647OpenFOAMPatchpublic2021-03-24 11:11
ReporterJuhoAssigned Towill 
Status resolvedResolutionfixed 
PlatformGNU/LinuxOSOtherOS Version(please specify)
Product Versiondev 
Fixed in Versiondev 
Summary0003647: multiphaseEulerFoam: linearTsub diameterModel does not find the saturationModels
DescriptionlinearTsub diameterModel still has the old style global saturation model lookup instead of the phase pair specific saturation models used currently. Thus, the saturationModel is not found and diameter is not updated during the simulations.

Modified linearTsubDiameter attached. Changes:
- Updated to use the current phase pair specific saturation models.
- Changed subcooling sign convention to to be compatible with the default parameters
- Added default values to the read function as well
- Print selected parameters to log
TagsNo tags attached.



2021-03-24 08:48


linearTsubDiameter.C (4,884 bytes)


2021-03-24 09:20

manager   ~0011951

Thanks for the patch, Juho. Small suggestion. Is it possible to use the phase system's lookup instead? It would save doing the name manipulations manually. E.g.:

    if (phase().fluid().foundSubModel<saturationModel>(pair))
        const saturationModel& satModel =


I'd try it myself, but I don't have a test case with this model active. I assume you do.


2021-03-24 09:43

reporter   ~0011952

Updated file attached. Much neater this way.

linearTsubDiameter-2.C (4,684 bytes)


2021-03-24 11:11

manager   ~0011953

Thanks for that. Pushed as commit

Issue History

Date Modified Username Field Change
2021-03-24 08:48 Juho New Issue
2021-03-24 08:48 Juho File Added: linearTsubDiameter.C
2021-03-24 09:20 will Note Added: 0011951
2021-03-24 09:43 Juho File Added: linearTsubDiameter-2.C
2021-03-24 09:43 Juho Note Added: 0011952
2021-03-24 11:11 will Assigned To => will
2021-03-24 11:11 will Status new => resolved
2021-03-24 11:11 will Resolution open => fixed
2021-03-24 11:11 will Fixed in Version => dev
2021-03-24 11:11 will Note Added: 0011953