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.

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/feature python solver derivation #2262

Merged
merged 28 commits into from
Jun 6, 2018

Conversation

jcotela
Copy link
Member

@jcotela jcotela commented Jun 4, 2018

This makes the basic python solvers of the FluidDynamicsApplication derive from the PythonSolver class (that is, we bring the FluidDynamicsApplication up to date with #2068).

This is the initial phase for the cleanup of solvers missing from #871.

@rubenzorrilla I did not test the trilinos version of the Ausas solver (because I don't have any example ready), could you try this one? I think I have covered the rest.

philbucher
philbucher previously approved these changes Jun 5, 2018
Copy link
Member

@philbucher philbucher left a comment

Choose a reason for hiding this comment

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

looks ok to me


self._SetUpRestart()

if self.load_restart:
self.restart_utility.LoadRestart()
restart_utility = self._GetRestartUtility()
restart_utility.LoadRestart()
Copy link
Member

Choose a reason for hiding this comment

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

this is in the baseclass now

Also I am writing a process for saving restarts atm such that one does not have to manually include it in each stage

Copy link
Member Author

Choose a reason for hiding this comment

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

I think this will come in #2135? I'll update once that is merged, then.

self._set_buffer_size()
self.main_model_part.SetBufferSize(self.min_buffer_size)

if not self.model.HasModelPart(self.settings["model_part_name"].GetString()):
Copy link
Member

Choose a reason for hiding this comment

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

this can be removed after the model is updated by riccardo

self.main_model_part.AddNodalSolutionStepVariable(KratosMultiphysics.CONDUCTIVITY)
self.main_model_part.AddNodalSolutionStepVariable(KratosMultiphysics.SPECIFIC_HEAT)
self.main_model_part.AddNodalSolutionStepVariable(KratosFluid.HEAT_CAPACITY_RATIO)

self.main_model_part.AddNodalSolutionStepVariable(KratosMultiphysics.IS_STRUCTURE) ## ?
self.main_model_part.AddNodalSolutionStepVariable(KratosMultiphysics.BODY_FORCE)
self.main_model_part.AddNodalSolutionStepVariable(KratosMultiphysics.NODAL_H) ## ?
self.main_model_part.AddNodalSolutionStepVariable(KratosMultiphysics.NODAL_AREA) ## ?
self.main_model_part.AddNodalSolutionStepVariable(KratosMultiphysics.REACTION) #for momentum
self.main_model_part.AddNodalSolutionStepVariable(KratosFluid.REACTION_DENSITY) #for momentum
Copy link
Member

Choose a reason for hiding this comment

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

I'd take the chance to update the comments.

Copy link
Member Author

Choose a reason for hiding this comment

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

To be honest, I think some of the marked variables may in fact be safely removed, but there is no test, so I didn't want to do it in this PR.

Copy link
Member

@rubenzorrilla rubenzorrilla left a comment

Choose a reason for hiding this comment

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

Everything looks good to me (let me now when you remove the CheckSkinProcess to approve again).

@jcotela
Copy link
Member Author

jcotela commented Jun 5, 2018

@rubenzorrilla I did it while you were reviewing, so github belives you approved it already...

@jcotela jcotela merged commit 40baebb into master Jun 6, 2018
@jcotela jcotela deleted the fluid/feature-python-solver-derivation branch June 6, 2018 09:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants