Louis bergmann/vtk quadric decimation vol constraint

Basic content checks passed!

Branch-at: b93218aa
Acked-by: @kwrobot

Do: test

@cory-quammen @lorensen @LouisBergmann test added for quadric decimation volume metric

Do: test

Testing commands handed to buildbot.

Branch-at: b93218aa

Your merge request has been queued for testing. You may view the test results on CDash or Buildbot.

Branch-at: b93218aa

Nice test! My only comment is that the differences in the side-by-side results in the baseline image look fairly minor. I wonder if tilting the plane 45 forward would make the difference more apparent.

To create a bigger difference I would probably have to play around with the attribute data and the surface (curvature and such). It could be done but finding the time is hard now, and mostly I want to test the code path and ensure that it is doing something reasonable. Maybe Louis has a better suggestion for doing something quick...

Roger that. I modified the test slightly to crank down the noise. The improvement is more pronounced when the noise is small. I also added an actor with the original noisy full-resolution plane, on the left, and changed the camera elevation to make the difference in bump heights more visible.

Attached is a the modified test file and new baseline. @will.schroeder could you update this branch with these?

QuadricDecimation2.py

Thanks Cory, I'll give it a day or two in case Louis has something else in mind. I thought about loading in a real terrain file, etc. but we are all busy :-)

Added 1 commit:

• 189a6b8d - Better test image thanks Cory

Basic content checks passed!

Changes since last check: compare

Branch-at: 189a6b8d
Acked-by: @kwrobot

Your merge request has been queued for testing. You may view the test results on CDash or Buildbot.

Branch-at: 189a6b8d

@cory-quammen what you think about giving us a +2 ?

+2

Do: merge

Branch successfully merged and pushed!

mentioned in commit 0e06f86c

Status changed to merged