In 2012, an attempt to replace vtkMath::TwoPi() with 2.0*vtkMath::Pi()
was incorrect. After further investigation, the offending committer
was Bill Lorensen. Shame on you!

cd1c29e6 Clean up by destroying the vtkTkRenderWidget.
Acked-by: Kitware Robot <kwrobot@kitware.com>
Reviewed-by: Brad King <brad.king@kitware.com>
Merge-request: !2125

Currently, the only way for Python/Tk to force Tcl to delete the
vtkTkRenderWidget is to destroy() the widget.  This has to be done with
care, since the widget must hold the only reference to its vtkRenderWindow
at the time of its destruction.  If the widget is not destroyed, then a
memory leak results.

d1cfc837 improve the speed of building the IBO for polys
Acked-by: Kitware Robot <kwrobot@kitware.com>
Merge-request: !2118

177f37d8 Padding filenames for ExodusII writer based on number of processes
Acked-by: Kitware Robot <kwrobot@kitware.com>
Reviewed-by: Cory Quammen <cory.quammen@kitware.com>
Merge-request: !2119

bd53b8f6 Update vtkExternalOpenGLCamera to use new explicit proj property.
fc820483 Add option to set an explicit projection matrix.
Acked-by: Kitware Robot <kwrobot@kitware.com>
Merge-request: !2089

In order to read in ExodusII files written out in parallel we
need to pad the process ids with zeros so that all of them are
associated together.

There was a spot where an ivar could be
set without checking if it already had a value that
needed to be released.

The BinningFilter ivar may also  be in a reference
loop as the input to this filter. Have to report it.

The context was getting cleaned too often maybe.
This fix reverts one line from commit
https://gitlab.kitware.com/vtk/vtk/commit/
3a8e4497

This commit fixes issue

vtk/vtk#16876

and adds a test for that condition as well as
a few other cases that should work but could
cause problems.

This change could introduce other issues limited
to windows as the code is fairly complex and
the use cases vary widely. At some point the whole
process of creating windows and handling offscreen
drawing needs to be reworked. It is a mishmash of 20
years of code and overdue for a real workover for all
platforms. But this topic is intended to fix this one
case until we have time to really look at it and rework
it.

Currently the new test is only active for OpenGL2 and
Windows. OpenGL1 and other platforms has issues with
it that need to be fixed in the future. But for now this
change fixes the reported Windows issues and adds a
test to maintain that fix.

There were no C++ tests that directly tested this class, and this test
also covers the recently-discovered bug for TranformInputSamplingOff().

A simple omission in the code caused TransformInputSamplingOff() to produce
incorrect results.  The default is TransformInputSamplingOn(), and it seems
that no tests exercised the class with it off.

Fix some long standing issues with multisamples
and stereo capable window requests on Win32.

This fixes an MPI deadlock issue when LIC is used in parallel.

30dd43d7 use vtkMath::IsNan
Acked-by: Kitware Robot <kwrobot@kitware.com>
Merge-request: !2120

Removed doxygen deprecation comment on vtkImageData methods

Fixed clang warning about presence of doxygen deprecation comment with no corresponding compiler attribute (ie no VTK_LEGACY)

See merge request !2102

Building the IBO for polys takes extra time as we have
to check for degenerate cells. This topic makes that
process about twice as fast.

102168d7 tests with leaks were not failing
Acked-by: Kitware Robot <kwrobot@kitware.com>
Reviewed-by: Will Schroeder <will.schroeder@kitware.com>
Merge-request: !2110

739af7c9 ENH: Add ability to read only a layer given the layer index.
f76f3c98 ENH: Read variables from single level, layer midpoints and layer interface.
Acked-by: Kitware Robot <kwrobot@kitware.com>
Merge-request: !2083

I am not sure what change caused this, but
test with debug leaks are no longer reported
as failing. This change causes them to fail
when run with ctest.

4c2f6570 fix memory leax in vtkPlotParallelCoordinates
Acked-by: Kitware Robot <kwrobot@kitware.com>
Merge-request: !2116

75c00d57 memory leak in ExtractHierarchicalBins
Acked-by: Kitware Robot <kwrobot@kitware.com>
Merge-request: !2115

f1e7549d Fixed memory leak
Acked-by: Kitware Robot <kwrobot@kitware.com>
Merge-request: !2114

There was a spot where an ivar could be
set without checking if it already had a value that
needed to be released.