Skip to content
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

vtkPolyDataNormals - Cell Normals #2434

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
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.

@luistorrescisneros
Copy link

luistorrescisneros commented Aug 31, 2023

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.

@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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
released Automated label team: Scientific Computing type: feature request 💡 Desired capabilities / enhancements
Projects
None yet
4 participants