overhaul the LESStep demo and rename it to BackwardFacingStep#312
overhaul the LESStep demo and rename it to BackwardFacingStep#312muezabdalla wants to merge 5 commits into
BackwardFacingStep#312Conversation
| App.ActiveDocument.PhysicsModel.Time = 'Transient' | ||
| App.ActiveDocument.PhysicsModel.Turbulence = 'LES' | ||
| App.ActiveDocument.PhysicsModel.TurbulenceModel = 'kEqn' | ||
| App.ActiveDocument.InitialiseFields.PotentialFlow = False |
There was a problem hiding this comment.
Initialisation of p and U do not seem to be set up?
There was a problem hiding this comment.
I think you will still need to set
App.ActiveDocument.InitialiseFields.UseOutletPValue = True
like in the original macro
| dimensions [0 2 -2 0 0 0 0]; | ||
|
|
||
| internalField uniform 0.0; | ||
| internalField uniform 83333.33333333334; |
There was a problem hiding this comment.
Thank you. Do the tests pass though? This seems like a strange p value...
There was a problem hiding this comment.
I am sorry for that I did not run the test because I thought this will not affect it and I was in a a hurry. but now I fixed it with the other typos and errors.
| App.ActiveDocument.InitialiseFields.Ux = '0.0 mm/s' | ||
| App.ActiveDocument.InitialiseFields.Uy = '0.0 mm/s' | ||
| App.ActiveDocument.InitialiseFields.Uz = '0.0 mm/s' | ||
| APP.ActiveDocument.InitialiseFields.BoundaryP = FreeCAD.ActiveDocument.CfdFluidBoundary001 |
There was a problem hiding this comment.
Typo in 'APP'? Have you tested this?
There was a problem hiding this comment.
CfdFluidBoundary001 doesn't yet exist - only created in part 04 - so I think the setting of the initialisation should be moved to after the BCs.
| App.ActiveDocument.PhysicsModel.Time = 'Transient' | ||
| App.ActiveDocument.PhysicsModel.Turbulence = 'LES' | ||
| App.ActiveDocument.PhysicsModel.TurbulenceModel = 'kEqn' | ||
| App.ActiveDocument.InitialiseFields.PotentialFlow = False |
There was a problem hiding this comment.
I think you will still need to set
App.ActiveDocument.InitialiseFields.UseOutletPValue = True
like in the original macro
2 participants
I kept all the parameters the same except the
startFrom(I changed it fromlatestTimetostartTime)