Removes all of the from __future__ imports that open and close the
brackets on the same line. This is most of them.

Runs clang-tidy pass by ref across the codebase to fix-up various
warnings we are seeing in cppcheck

Signed-off-by: Steven Hahn <hahnse@ornl.gov>

Updates the CMake configuration to set our debugging targets. This means
a user can debug Workbench or System tests, which have a python entry
point, directly from the MSVC IDE.

Also updated class_maker.py

Fixes all flake8 3.7.9 warnings as part of the Python 2->3 transition.

This commit replaces all header guards in qt/icons and qt/paraview_ext with #pragma once.

This commit replaces all header guards in qt/scientific_interfaces with #pragma once.

This commit replaces all header guards in qt/Widgets with #pragma once.

We want plot windows to remain on top of their parent.
The old method of forcing the figure window to the top
when mainwindow gained focus ends up causing a fight with
the window manager over what has focus. Some window
managers, GNOME, allow the script editor to have focus, others
such as Xfce do not. This is a compromise as we lose the
minimize button (but not on Windows) yet the window is
more likely to play correctly with the rest of the ecosystem.

Different flags/parent behaviour is set per OS as each handles
this quite differently. The downside on some Linux window managers
is that the figures are now missing a minimize button. The visible
button in the Plots toolbox still works though.
Refs #27883

This updates the test to check that the new return values of replaceWorkspaceArtists and removeWorkspaceArtists return correctly.

Previously on workbench the PreviewPlot mpl implementation was re-drawing every axes when any workspace changed. This was leading to the time taken to respond to ADS changed signals being high.

Fixes the progress arrow which previously only pointed at the first
line, irrespective of selection. Additionally removes a method which is
completely unused in the code-base, but looks like it would do a similar
thing as it's misleading.

A struct is used in case we need to add additional fields, and for the
extra readability when we dereference the map iterator. This avoids a
iter->value.at<0> in favour of having named fields of a struct.