[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Getfem-commits] merge request for touched_region_for_projected_fem

From: Konstantinos Poulios
Subject: Re: [Getfem-commits] merge request for touched_region_for_projected_fem
Date: Thu, 4 Mar 2021 12:31:21 +0100

Hi Andriy,

Thanks, I see how it can be useful. Could I ask you to reduce the use of auto for this commit? For example it does not make much sense to use auto for bool. In general my preference for the GetFEM codebase is to use auto only if some type is particularly long and makes the code significantly less readable. Otherwise the type of the variables is useful information for people that will read and try to understand the code later.

There is also a typo in a comment. It should be "Gauss".

Best regards

On Thu, Mar 4, 2021 at 11:32 AM Andriy Andreykiv <> wrote:
Dear Yves and Konstantinus,

Kind request to review and merge touched_region_for_projected_fem branch.
It introduces a method for projected_fem that extracts a region from the target that is actually touched by the source.
I use this region to integrate my mortar terms on.

Best regards,

reply via email to

[Prev in Thread] Current Thread [Next in Thread]