CfdOF default maxCo/maxAlphaCo in controlDict

A subforum specific to the development of the OpenFoam-based workbenches ( Cfd https://github.com/qingfengxia/Cfd and CfdOF https://github.com/jaheyns/CfdOF )

Moderator: oliveroxtoby

User avatar
markinca
Posts: 56
Joined: Sun Apr 30, 2023 2:41 pm

Re: CfdOF default maxCo/maxAlphaCo in controlDict

Post by markinca »

I can fully appreciate the desire to make things simple. However, for transient analyses, I think it is very difficult to find a set of inputs that will work with a large range of scenarios. At least from my experience, there is nearly always a need for some customization and optimization. Having a wider range of parameters available if only in just the properties window would be very helpful. Thanks for the feedback.
User avatar
oliveroxtoby
Posts: 837
Joined: Fri Dec 23, 2016 9:43 am
Location: South Africa

Re: CfdOF default maxCo/maxAlphaCo in controlDict

Post by oliveroxtoby »

markinca wrote: Tue May 16, 2023 3:14 pm I can fully appreciate the desire to make things simple. However, for transient analyses, I think it is very difficult to find a set of inputs that will work with a large range of scenarios. At least from my experience, there is nearly always a need for some customization and optimization. Having a wider range of parameters available if only in just the properties window would be very helpful. Thanks for the feedback.
I don't disagree with that. It's should be quite straightforward to add those you want by following what I did in this commit (then please issue a pull request to get them merged).
User avatar
markinca
Posts: 56
Joined: Sun Apr 30, 2023 2:41 pm

Re: CfdOF default maxCo/maxAlphaCo in controlDict

Post by markinca »

Thanks!
Post Reply