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

Openmesh interface #36

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

aschier
Copy link

@aschier aschier commented Mar 25, 2019

I added a simple OpenMesh interface to integrate BFF with other programs.

It currently generates face-vertex UVs stored at the halfedge of a triangle incident to the vertex and does not try to preserve UV shells, but it is planned to do so.

Support for polygons would be possible, but is currently out of scope in our projects. Currently it can possibly be added rather easy as a wedge corresponds to a halfedge of the polygon and maybe we will include it later.

As triangle structure it depends on https://github.com/aschier/openmesh_structures for mesh traits with double precision, so we avoid using templates and BFF can be installed as a library.

@aschier
Copy link
Author

aschier commented Mar 25, 2019

The other commits:

  • Fix for the invalid end vector iterator (fixes Iterator problem in MeshIO.buildMesh #29).
  • Fix for missing algorithm include (needed at least for MSVC).
  • Improved CMake files:
    • Create a library.
    • Provide exported targets.
    • Install targets.
    • Add headers to the targets, so they get listed in cmake output formats which list project files, e.g., Visual Studio projects.
  • Added another FindSuiteSparse.cmake file, which works (better) for me. You may need to test if it works for you as well.

@aschier aschier force-pushed the openmesh-interface branch from 452da34 to adb5151 Compare April 29, 2019 08:58
@aschier
Copy link
Author

aschier commented Apr 29, 2019

I added another commit, that removes the dependency on an external common Traits class by using templates. I currently added a template for the traits and not the mesh, as the interface is only tested for triangle meshes anyway.

@aschier aschier force-pushed the openmesh-interface branch from adb5151 to 2ccdfcf Compare May 5, 2020 10:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant