Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in / Register
  • iMSTK iMSTK
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 83
    • Issues 83
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 14
    • Merge requests 14
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • iMSTKiMSTK
  • iMSTKiMSTK
  • Issues
  • #367
Closed
Open
Issue created Sep 13, 2021 by Andrew Wilson@andrew.wilson🐘Contributor

Rename SurfaceMesh::flipNormals to SurfaceMesh::reverse, or SurfaceMesh::reverseWinding

Simple change, reminder for later. The issue with flip normals is that it doesn't actually flip normals. It reverses windings.

In fact if you have per vertex normals defined and call flipNormals, expecting the normals to flip. It does nothing. Because again, it doesn't flip normals, it reverses windings. Two very different things.

Assignee
Assign to
Time tracking