-
Notifications
You must be signed in to change notification settings - Fork 4
Mesh geometry utils #135
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: develop
Are you sure you want to change the base?
Mesh geometry utils #135
Conversation
e83632c
to
671e8f1
Compare
closes #117 |
…o mesh-geometry-utils-117
…o mesh-geometry-utils-117
…esh-geometry-utils-117
…o mesh-geometry-utils-117
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@pabloyesteb also ensure that your new functions within the Mesh
class are properly documented. We will improve the documentation of the whole class soon.
@pabloyesteb also check that the new implementation of Once these check out, I'll finish implementing the compiled versions of these functions and we will be ready for a final test and merge. |
I've checked the edge_normals implementation in ParaView and everything is alright. Also, I've checked the documentation of my functions in mesh.py and everything seems to be in order to me. ¿Do you have any specific suggestions? |
…o mesh-geometry-utils-117
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@pabloyesteb can you check that the new compiled version of your routines produce the same expected result as the non compiled version?
|
||
@cr('math.fix_coherence') | ||
def fix_normals_coherence(normals, connectivity): | ||
''' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fix docstring
@cython.cdivision(True) # turn off zero division check | ||
def fix_normals_coherence(real[:,:] normals, object edge_dict, object adjacency, int num_cells): | ||
''' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fix docstring
Implemented functions for computing the cells connectivity and edge normals. Added an example. The functions have been ckecked to properly compute the claimed data in ParaView.