TUT: clean up multiphase tutorials
Edited by Kutalmış Berçin
Merge request reports
Activity
Filter activity
changed milestone to %v2006
- Resolved by Kutalmış Berçin
- Resolved by Kutalmış Berçin
- Resolved by Kutalmış Berçin
@kuti - Nice to see this clean-up. Lots of work there.
Didn't check for it in the changes - do the PDRFoam tutorials now use the update thermo specs?
- Resolved by Kutalmış Berçin
- Resolved by Kutalmış Berçin
@kuti - looks good
- General rule for setting
purgeWrite
to 10? - Need to check for possible regressions before merge
- General rule for setting
- Resolved by Kutalmış Berçin
General rule for setting purgeWrite to 10?
The sole motivation was to manage disk space in a more economic way (e.g. I'm not able to run the multiphase tutorials in my workstation due to exceeding disk space). Yet I think 10 might be very low for some cases.
Might consider a higher cap, 50? 100? - or reinstantiate to 0?
- Resolved by Kutalmış Berçin
Probably leave as-is for now.
added 35 commits
-
55f0a0c4...68b2f503 - 34 commits from branch
develop
- 8212edcb - TUT: clean up multiphase tutorials
-
55f0a0c4...68b2f503 - 34 commits from branch
good to go @andy, if @Prashant gives the green light.
- I verified bitwise regressions for all the changes (except one case where the tutorial was not executing the main application at all, I enabled the solver).
- I have moved some of the files in and out TUT/resources, so sourcing etc/bashrc for an existing installation should be problematic since old compilations will not find the resource file paths required by some of the tutorials.
- The same OF compilation should be used to test for the branch with and without the commit since some changes have been made in multiphase tutorials during the last three weeks.
- There might still be something that I have missed, sorry for them - though chances are lowered.
Thank you!
Edited by Kutalmış Berçin
Please register or sign in to reply