Kratos: [Core][ConstitutiveLaw] TransformStrains is not a virtual method

Created on 5 Sep 2018  路  5Comments  路  Source: KratosMultiphysics/Kratos

In line 1010 of constitutive_law.h the method TransformStrains is not declared virtual, but the method TransformStresses it is. Is this right?

https://github.com/KratosMultiphysics/Kratos/blob/f86de29047f9cd7dc92c0cc95c299a4365948b3a/kratos/includes/constitutive_law.h#L1010

https://github.com/KratosMultiphysics/Kratos/blob/f86de29047f9cd7dc92c0cc95c299a4365948b3a/kratos/includes/constitutive_law.h#L1023

Kratos Core Question
Source
picture loumalouomega
👍2

Most helpful comment

In fact these methods are utilities that would be better to be located in another place (i.e. constitutive_law_utilities) non virtual.

picture josep-m-carbonell on 5 Sep 2018
👍2

All 5 comments

@KratosMultiphysics/technical-committee @KratosMultiphysics/structural-mechanics @KratosMultiphysics/solid-mechanics

loumalouomega picture loumalouomega on 5 Sep 2018

In fact these methods are utilities that would be better to be located in another place (i.e. constitutive_law_utilities) non virtual.

josep-m-carbonell picture josep-m-carbonell on 5 Sep 2018
👍2

I agree with that, besides that would make the CL lighter (which is something everyone will agree)

loumalouomega picture loumalouomega on 5 Sep 2018

@KratosMultiphysics/technical-committee: The less intrusive option would be to add the virtual to the CL
base method.

pooyan-dadvand picture pooyan-dadvand on 19 Oct 2018
👍1

Added

loumalouomega picture loumalouomega on 19 Oct 2018
Was this page helpful?
0 / 5 - 0 ratings

Related issues

maceligueta picture maceligueta  路  6Comments
KlausBSautter picture KlausBSautter  路  6Comments
ipouplana picture ipouplana  路  7Comments
rubenzorrilla picture rubenzorrilla  路  4Comments
e-dub picture e-dub  路  3Comments