[Octave-bug-tracker] [bug #55642] isosurface is slow

[Guillaume]

Follow-up Comment #3, bug #55642 (project octave):

Thanks Michael, the speed improvement is impressive! To be fair, the speed
issue was acknowledged by the authors in the FIXME note in
__unite_shared_vertices__.m and it was probably not so important for small
examples.

A few comments/questions:

* __unite_shared_vertices__.m probably still has to be kept as it is also used
by reducepatch.m (patch #8912).

* Concerning tolerance, this was discussed in bug #46946, see Marco's comment
#8. This would make the tolerance a vector instead of a scalar in the loop?

* The one line I don't understand is this one. Why is it enough to only
compare with a [2 3 1] reordering?


fvc.faces = fvc.faces(all (fvc.faces-fvc.faces(:,[2 3 1])!=0,2),:);


    _______________________________________________________

Reply to this item at:

  <https://savannah.gnu.org/bugs/?55642>

_______________________________________________
  Message sent via Savannah
  https://savannah.gnu.org/