Code comment on incremental elastic energy (!17) · Merge requests · mpm-at-recover / cbgeo · GitLab

Jerome Duriez requested to merge workComments into custom_cbgeo Nov 06, 2023

Sort-of follow-up of !8 (merged). Sacha @sacha.duverger please check comments are ok and let us discuss https://forgemia.inra.fr/mpm-at-recover/cbgeo/-/blob/fc04e57ab6625672efbbd63c484bd9ec2baba1c5/include/materials/mohr_coulomb.tcc#L497-499 (= first commit here) which seems to be incorrect to me. If .cwiseProduct(strain_coefs) is to be used L498, it should apply to deps_el (which is for now missing the "2" factor), not to dstrain (which already has the "2" factor), no ?