getfem-commits
[Top][All Lists]
Advanced

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

Re: [Getfem-commits] please merge branch devel-tetsuo-add_lumped_mass_py


From: Konstantinos Poulios
Subject: Re: [Getfem-commits] please merge branch devel-tetsuo-add_lumped_mass_python_interface_squash
Date: Fri, 9 Apr 2021 21:24:34 +0200

Dear Tetsuo and Yves

@Tetsuo thanks for this interface functions. I would like to use this opportunity to discuss about the word "brick" in the naming of our functions adding PDE term matrices. Traditionally, "brick" was only used at the end of the function name. More recently, Yves started skipping "brick" from some of the newer terms, which I think is fine. Now we have a mix with names including "brick" and some without. Apart from that, there are 3 functions that do not follow the old naming convention at all:
add_isotropic_linearized_elasticity_brick_pstrain
add_isotropic_linearized_elasticity_brick_pstrain
add_lumped_mass_brick_for_first_order
I would like to avoid this kind of inconsistencies in the naming so that all functions for adding a PDE matrix either have "brick" at the end of their name or that they do not include it at all.

For Tetsuo's function I would suggest one of
add_lumped_mass_matrix_for_first_order
add_lumped_mass_for_first_order_brick

Both in c++ and the interfaces.

What do you think?

Best regards
Kostas

On Fri, Apr 9, 2021 at 11:23 AM Tetsuo Koyama <tkoyama010@gmail.com> wrote:
Dear GetFEM project

I added the lumped mass for first order to the python interface.
Could you merge devel-tetsuo-add_lumped_mass_python_interface_squash ?

BR Tetsuo

reply via email to

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