Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Fluid] Fine grained Python solvers #6575

Merged
merged 48 commits into from
Mar 31, 2020
Merged

[Fluid] Fine grained Python solvers #6575

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
48 commits
Select commit Hold shift + click to select a range
96e5fb4
Base solver
rubenzorrilla Mar 14, 2020
e2ab850
Working monolithic and FS solvers
rubenzorrilla Mar 14, 2020
d5035d1
Serial solvers working
rubenzorrilla Mar 15, 2020
ac17ab4
Remove old steady solver
rubenzorrilla Mar 15, 2020
2caacb2
Serial solvers clean up
rubenzorrilla Mar 16, 2020
889a4f2
MPI solvers (testing is pending)
rubenzorrilla Mar 16, 2020
aec2a26
Merge branch 'master' into fluid/fine-grained-initialize
rubenzorrilla Mar 18, 2020
17159d8
Adding missing Trilinos scheme
rubenzorrilla Mar 18, 2020
b98d832
Fix name mangling
rubenzorrilla Mar 19, 2020
3014ff8
Minor
rubenzorrilla Mar 19, 2020
e06ca98
Using AMGCL as default linear solver
rubenzorrilla Mar 19, 2020
7ff8119
Using self.__class__.__name__
rubenzorrilla Mar 19, 2020
11fc9ab
Adjoint solvers
rubenzorrilla Mar 20, 2020
aebc95d
Chimera solvers
rubenzorrilla Mar 20, 2020
bd58e39
Minor correction
rubenzorrilla Mar 20, 2020
255f2cc
Potential flow solvers
rubenzorrilla Mar 20, 2020
9e3900e
Using getters, main model part and adding missing initialize
marcnunezc Mar 20, 2020
6ba1b9c
Merge branch 'master' into fluid/fine-grained-initialize
rubenzorrilla Mar 20, 2020
a7152df
Adding @classmethod decorators to potential flow solvers
rubenzorrilla Mar 20, 2020
7f4ab1a
changes to chimera app to solve test failures
adityaghantasala Mar 20, 2020
060d974
add detailed optional output
adityaghantasala Mar 20, 2020
6400308
Merge branch 'fluid/fine-grained-initialize' of https://github.com/Kr…
adityaghantasala Mar 20, 2020
e69bc08
codecy spaces
adityaghantasala Mar 20, 2020
a82e8fe
Using computing_model_part and adapting test jsons
marcnunezc Mar 20, 2020
7f4d93e
Merge branch 'fluid/fine-grained-initialize' of https://github.com/Kr…
marcnunezc Mar 20, 2020
414db10
Merge branch 'master' into fluid/fine-grained-initialize
rubenzorrilla Mar 21, 2020
a8c6105
solution_scheme to scheme
rubenzorrilla Mar 21, 2020
96c1616
Update applications/FluidDynamicsApplication/python_scripts/fluid_sol…
rubenzorrilla Mar 21, 2020
8f06292
Merge branch 'master' into fluid/fine-grained-initialize
rubenzorrilla Mar 22, 2020
da89039
FM-ALE DOFs message
rubenzorrilla Mar 22, 2020
d1d5144
Follow style guide
rubenzorrilla Mar 22, 2020
78be497
More code review and Codacy
rubenzorrilla Mar 22, 2020
f0e3e1c
Update applications/FluidDynamicsApplication/python_scripts/trilinos_…
rubenzorrilla Mar 23, 2020
9dddf45
Update applications/FluidDynamicsApplication/python_scripts/trilinos_…
rubenzorrilla Mar 23, 2020
3b44eec
Update applications/FluidDynamicsApplication/python_scripts/trilinos_…
rubenzorrilla Mar 23, 2020
000dca5
Merge branch 'master' into fluid/fine-grained-initialize
rubenzorrilla Mar 23, 2020
d8be320
change mpi solver tolerance
sunethwarna Mar 23, 2020
726aefc
Merge branch 'fluid/fine-grained-initialize' of https://github.com/Kr…
sunethwarna Mar 23, 2020
95cf8c8
adding comment
sunethwarna Mar 23, 2020
017d37f
Check restart
rubenzorrilla Mar 23, 2020
110943b
Merge branch 'master' of https://github.com/KratosMultiphysics/Kratos…
adityaghantasala Mar 23, 2020
d59aeeb
Merge branch 'master' into fluid/fine-grained-initialize
rubenzorrilla Mar 27, 2020
b7b6858
@philbucher suggestions
rubenzorrilla Mar 27, 2020
d758736
Missing aux Darcy test solver change
rubenzorrilla Mar 27, 2020
6213eee
Merge branch 'master' into fluid/fine-grained-initialize
philbucher Mar 29, 2020
339612d
fixing NavierStokesAleFluidSolver
philbucher Mar 29, 2020
2f7b4ca
Potential flow style guide
rubenzorrilla Mar 30, 2020
f65a3ae
Merge branch 'fluid/fine-grained-initialize' of https://github.com/Kr…
rubenzorrilla Mar 30, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 4 additions & 1 deletion ...ications/ChimeraApplication/python_scripts/navier_stokes_solver_fractionalstep_chimera.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,7 @@ def AddVariables(self):
self.main_model_part.AddNodalSolutionStepVariable(KratosChimera.ROTATION_MESH_DISPLACEMENT)
self.main_model_part.AddNodalSolutionStepVariable(KratosChimera.ROTATION_MESH_VELOCITY)

KratosMultiphysics.Logger.PrintInfo(self.__class__.__name__, "Fluid solver variables added correctly.")
KratosMultiphysics.Logger.PrintInfo(self.__class__.__name__, "Fluid chimera solver variables added correctly.")

def ImportModelPart(self):
if(self.settings["model_import_settings"]["input_type"].GetString() == "chimera"):
Expand Down Expand Up @@ -60,6 +60,9 @@ def Initialize(self):
self.model,
self.chimera_internal_parts)

def GetComputingModelPart(self):
return self.main_model_part

def InitializeSolutionStep(self):
self.chimera_process.ExecuteInitializeSolutionStep()
super(NavierStokesSolverFractionalStepForChimera,self).InitializeSolutionStep()
Expand Down
9 changes: 6 additions & 3 deletions applications/ChimeraApplication/python_scripts/navier_stokes_solver_vmsmonolithic_chimera.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ class NavierStokesSolverMonolithicChimera(NavierStokesSolverMonolithic):
def __init__(self, model, custom_settings):
[self.chimera_settings, self.chimera_internal_parts, custom_settings] = chimera_setup_utils.SeparateAndValidateChimeraSettings(custom_settings)
super(NavierStokesSolverMonolithicChimera,self).__init__(model,custom_settings)
KratosMultiphysics.Logger.PrintInfo(self.__class__.__name__, "Construction of NavierStokesSolverMonolithic finished.")
KratosMultiphysics.Logger.PrintInfo(self.__class__.__name__, "Construction of NavierStokesSolverMonolithicChimera finished.")

def AddVariables(self):
super(NavierStokesSolverMonolithicChimera,self).AddVariables()
Expand All @@ -28,7 +28,7 @@ def AddVariables(self):
self.main_model_part.AddNodalSolutionStepVariable(KratosChimera.ROTATION_MESH_DISPLACEMENT)
self.main_model_part.AddNodalSolutionStepVariable(KratosChimera.ROTATION_MESH_VELOCITY)

KratosMultiphysics.Logger.PrintInfo(self.__class__.__name__, "Fluid solver variables added correctly.")
KratosMultiphysics.Logger.PrintInfo(self.__class__.__name__, "Fluid chimera solver variables added correctly.")

def ImportModelPart(self):
if(self.settings["model_import_settings"]["input_type"].GetString() == "chimera"):
Expand Down Expand Up @@ -57,6 +57,9 @@ def Initialize(self):
self.model,
self.chimera_internal_parts)

def GetComputingModelPart(self):
return self.main_model_part
Copy link
Member Author

@rubenzorrilla rubenzorrilla Mar 21, 2020 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@adityaghantasala Just for me to know. Why do you need the main_model_part? It is because it contains both the background mesh and the patches?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

not exactly. I need to use the main_model_part because the constraints are added on this modelpart and computing_model_part does not know anything about them.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

mmmm now I see. Do you think we should add them to the computing model part too?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In the current impelementations, not necessary. But as I remember that computing modelpart is set to be removed in future no ?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In the structure. It is required to have it in the fluid.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is it really required in FluidApp ? If not we can also try to remove it !

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I discussed this many times with @jcotela, he said it is not possible to remove it
Not sure if this has changed


def InitializeSolutionStep(self):
self.chimera_process.ExecuteInitializeSolutionStep()
super(NavierStokesSolverMonolithicChimera,self).InitializeSolutionStep()
Expand All @@ -66,7 +69,7 @@ def FinalizeSolutionStep(self):
## Depending on the setting this will clear the created constraints
self.chimera_process.ExecuteFinalizeSolutionStep()

def _create_builder_and_solver(self):
def _create_builder_and_solver(self):
linear_solver = self.get_linear_solver()
if self.settings["consider_periodic_conditions"].GetBool():
KratosMultiphysics.Logger.PrintInfo("NavierStokesSolverMonolithicForChimera Periodic conditions are not implemented in this case .")
Expand Down