yade-users team mailing list archive

[Jerome Duriez <duriez@xxxxxxxxxxxxxxx>]

Bruno Chareyre a écrit :
> Jerome Duriez a écrit :
>   
>> Hello everybody. I just noticed that the class 
>> "CohesiveFrictionalContactInteraction" (in pkg/dem/Engine/EngineUnit/) 
>> inherits from the "_ElasticContact_Interaction", whereas in the 
>> registerAttributes (of CohesiveFrictionalContactInteraction) what is 
>> called first is :
>> _SimpleElastic_Interaction::registerAttributes();
>>
>> It seems to me that it is quite not logic...
>> In fact some of the attributes which belong to 
>> CohesiveFrictionalContactInteraction through ElasticContactInteraction 
>> (but which are not in SimpleElasticInteraction) are however registered 
>> (like tangensofFrictionAngle for exemple), but the work is not perfectly 
>> done : for example "ks" is not registered.
>>   
>>     
> Thank you for the remark. I added "ks" in registered parameters. It 
> should be enough to fix the problem, shouldn't it?
> There is no real reason to consider this a bug. Registering 
> ElasticContactIntercations would need extra disk space for variables 
> like "equilibriumDistance", which are in fact useless in the current 
> cohesive-frictional law.
>   
Ok, that's in fact a detail, but so why making 
CohesiveFrictionalContactInteraction inherit from 
ElasticContactInteraction if not all its arguments are used ? To not 
define again an other class which would be "intermediate" between 
SimpleElasticInteraction and ElasticContactInteraction ? (this could be 
a reason, indeed)
Thank your for your response (and for the commit...)

Jerome

_______________________________________________
Yade-users mailing list
Yade-users@xxxxxxxxxxxxxxxx
https://lists.berlios.de/mailman/listinfo/yade-users