Skip to content

vtkPolyDataNormals - Cell Normals #2434

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom
Closed
emil-peters opened this issue May 24, 2022 · 3 comments · Fixed by #2912 · May be fixed by #2436
Closed

vtkPolyDataNormals - Cell Normals #2434

emil-peters opened this issue May 24, 2022 · 3 comments · Fixed by #2912 · May be fixed by #2436
Assignees
@emil-peters
Labels
released Automated label team: Scientific Computing type: feature request 💡 Desired capabilities / enhancements

Comments

@emil-peters
Copy link
Contributor

Need

At the moment, the vtkPolyDataNormals is only able to calculate the vertex normals.
I might be interesting to also compute the cell normals instead based on a programmatically set boolean.

Based on the suggestion of @finetjul in this reply

ps. You can assign me on this issue.
@emil-peters emil-peters added the type: feature request 💡 Desired capabilities / enhancements label May 24, 2022
emil-peters added a commit to emil-peters/vtk-js that referenced this issue May 25, 2022
@emil-peters
feat(polydatacellnormals): programatically choose to compute cell nor…
dd32dfc 
…mals

closes Kitware#2434
@emil-peters emil-peters mentioned this issue May 25, 2022
Draft
3 tasks
@luistorrescisneros
Copy link

luistorrescisneros commented Aug 31, 2023 •
edited
Loading

I will use this spot to draw my concern cause it is somehow related. How it is posible to a attach a normal vector to a single point? I came across it, when using vtkPolyDataNormals to compute the normals of some cells, since a cell in some situations represents a surface. Basic mathematics teach us a normal vector can be computed solely on N-Dimensional (where N>2) surfaces. In other words, not on single points or even lines.

@finetjul
Copy link
Member

finetjul commented Sep 1, 2023

A point typically has the same normal of the cell.
I agree it is usually more useful to have cell normals than point normals.

finetjul added a commit to finetjul/vtk-js that referenced this issue Sep 9, 2023
@finetjul
feat(polydatanormals): add option to compute cell normals
5a6a6cd 
fix Kitware#2434
@finetjul finetjul mentioned this issue Sep 9, 2023
Merged
5 tasks
finetjul added a commit to finetjul/vtk-js that referenced this issue Sep 17, 2023
@finetjul
feat(polydatanormals): add option to compute cell normals
7e224f7 
fix Kitware#2434
github-merge-queue bot pushed a commit that referenced this issue Sep 20, 2023
@finetjul @floryst
feat(polydatanormals): add option to compute cell normals
602c9e8 
fix #2434
@github-actions
Copy link

🎉 This issue has been resolved in version 28.12.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@github-actions github-actions bot added the released Automated label label Sep 20, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees

@emil-peters emil-peters

Labels
released Automated label team: Scientific Computing type: feature request 💡 Desired capabilities / enhancements
Projects
None yet
Milestone
No milestone
4 participants
@finetjul @jourdain @emil-peters @luistorrescisneros