[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Getfem-commits] Could you merge 4 branchs
From: |
Tetsuo Koyama |
Subject: |
[Getfem-commits] Could you merge 4 branchs |
Date: |
Sun, 23 Feb 2020 17:29:56 +0900 |
Dear getfem proejct
Could you review and merge 4 branchs?
1 is API adding and 3 is very small fix.
1. devel-tetsuo-outer_faces_in_ball
I added outer_faces_in_ball to python API and select_faces_in_ball to C++ API.
It makes easy to select region in ball shape.
I know we can select convex and faces by using numpy, but this API
save that work.
(I really like GetFEM++'s mesh API.)
2. devel-tetsuo-cleanup
Variable 'is_set' is assigned a value that is never used. I deleted
it in interface/src/getfemint_levelset.cc .
3. tetsuo-comment-of-pts-from-cvid
I made a small fix of message in gmm to show me what is the wrong of use.
4. tetsuo-comment-of-pts-from-cvid
My friend @phduckie said that in GetFEEM++ Python API document it is
not clear that what is returned from function. So I changed the
comment of function a little which is used for documentation.
Thank @phduckie for your comment.
https://twitter.com/phduckie/status/1217557364204699649
Best Regard Tetsuo
- [Getfem-commits] Could you merge 4 branchs,
Tetsuo Koyama <=