Integer-only cross field computation

SIGGRAPH 2018


Reviews

Information

  • Paper topic: Geometry
  • Software type: Code
  • Able to run a replicability test: True
  • Replicability score: 5
  • Software language: Matlab / Mathematica / .., Shader language
  • License: unspecified
  • Build mechanism: Other script, Not applicable (python, Matlab..)
  • Dependencies: matlab / libigl
  • Documentation score {0,1,2}: 2
  • Reviewer: Nicolas Bonneel <nicolas.bonneel@liris.cnrs.fr>
  • Time spent for the test (build->first run, timeout at 100min): 40min

Source code information

Comments

Minor fixes to the code: when running the examples from the examples directory, you need to replace './data/bunny.off' by '../data/bunny.off' in the demo files, and similarly change the '.' to '..' in the vis_vector_field.m file in addOptional(p, 'BinaryPath', fullfile('..', 'bin', 'vis_vector_field', 'vis_vector_field_bin.exe')); 
I also had to add my visual studio bin directory to my PATH.
Also, the readme mentions a code snippet to compute cross fields. There is an error in the routine to display the result : it is not m.draw(CF, alpha) but m.plot(CF, alpha) (the method draw is not a method of the class Mesh, and 'draw_face_field' is not correct either).
A single example (bunny) is provided, though I believe it should work similarly well on other examples.
It took me significant time making this code run only because I had to perform a fresh cuda install -- it otherwise trivially installs.

If you want to contribute with another review, please follow these instructions.

Please consider to cut/paste/edit the raw JSON data attached to this paper.