GitLab

Registered testing thread and OpenGL context with OpenGLJobProcessor.

* Updated DevilImageWriter, added property to make writing depth images optional.

initial py script. given 3 directory compares reference and testruns to store results

Async signals

See merge request !87

refs #384

* Slight changes to the API: renamed signal::trigger() to signal::triggerSignal() and signal::queue() to signal::queueSignal()
* Replaced all sigslot signal emits through operator() with emits through emitSignal() to enable debug feature.
* Fixed a possible race condition when deleting a GeometryTransferFunction and its editor window at the same time (as this will happen from different threads).

refs #384

When compiled with CAMPVIS_DEBUG, all signals carry additional information on the calling function, file and line. This is realized transparently through some macro magic when client code emits signals through the new emitSignal() method. Unfortunately, there is no way to debug-overload the operator(), hence, while this way of emitting signals remains valid, it cannot write the debug information into the signal handle.

To avoid symbol conflicts, _connection_base::emitSignal() and has_slots::emitSignal() had to be renamed to ::processSignal().

refs #384

Introducing SignalHandlingMode to signal_manager to allow forcing all signals being handled directly (needed for test-campvis) or queued.

refs #384

refs #384

Itksegmentation rescale

1. Rescaling was needed for restricting the intensity values of images used in segmentation to [0,255]. Some images were exceeding this range.

2. Corrected copyright year range.

See merge request !86

Volume explorer mouse event - Y coordinates

Small correction:  Y coordinates of the render size should be used instead of X coordinates when subtracting from me->y().

See merge request !85

* Fixed crash (due to nullptr) in DataContainerInspectorWidget if DevIL module not present.

refs #384

* Updated sigslot::concurrent_pointer_list::insert() and sigslot::concurrent_pointer_list::push_back() to fill holes where items already have been deleted. This avoids degrading performance due to excessive fragmentation.
* Updated sigslot legal header.

Updated sigslot::concurrent_pointer_list implementation as well as added several typename declarations to calm down GCC.

Removed all occasions where references to DataHandles were used, since especially their use in signals may lead to segfaults or other undesired behavior.

Removed unlockInExtraThread flag from AbstractProcessor::process() - this is no longer needed as signal handling is done asynchroneously now anyway.

#ref 384

Conflicts:
	core/pipeline/abstractprocessor.cpp

Conflicts:
	core/pipeline/abstractprocessor.cpp
	core/pipeline/abstractprocessor.h

Introducing sigslot::concurrent_pointer_list<T> a list-like container allowing thread-safe bidirectional iteration, insertion and removal of elements. Single constraint is that the element type must be a pointer type (or at least have pointer semantics) and never be 0. This is neat, as this class offers exactly what we need for a lock-free sigslot implementation.

Updated sigslot implementation to use concurrent_pointer_list for connection storage. Hence, we can get rid of the locking mechanisms, which got completely removed.

Since this simplified template definitions for sigslot, many campvis files needed to be adapted (i.e. removing the <> from has_slots<> and signal0<>).

refs #384

Conflicts:
	core/classification/genericgeometrytransferfunction.h
	core/eventhandlers/trackballnavigationeventlistener.h
	core/pipeline/processordecoratormasking.h
	core/properties/propertycollection.h
	core/properties/transferfunctionproperty.h
	core/tools/opengljobprocessor.h

* Introduced check whether signal emitting thread is the signal manager thread. In such a case signals are now emitted synchronously per default.
* Fixed sigslot shared library builds

refs #384

Register std::vector<DataContainer*> and std::vector<AbstractPipeline*> as Qt meta types in mainwindow.cpp to support them in Qt signals (even though I do not understand why the issue did not arise before...)

* implemented signal_handle2 through signal_handl5
* removed all implementations of signals with more than 5 arguments (nobody needs that many args and all those template instantiations just kill compile time)

refs #384

Conflicts:
	ext/sigslot/sigslot.h

* Moved campvis::Runnable interface to tgt namespace (since it's needed by sigslot, which only depends on tgt)
* Introduced sigslot::signal_manager singleton class that will manage the dispatching of signals in its own thread
* Started proof-of-concept implementation of asynchroneous signals for signal0<> and signal1<>. Both classes define their own signal_handleN deriving from _signal_handle_base, which defines the signal to dispatch.

Proof-of-concept implementation seems to work so far.

refs #384

Conflicts:
	core/tools/opengljobprocessor.h
	ext/tgt/runnable.h

Conflicts:
	application/CMakeLists.txt
	core/tools/opengljobprocessor.h

Refactor various

If you need one of these two C++11 headers, include <ext/threading.h> instead, which will use C++11 headers if present or TBB's compatibility layer otherwise.

closes #567

This was due to a missing/wrong GL_PACK_ALIGNMENT setting.

closes #568

INVALID_RESULT, INVALID_PROPERTIES, INVALID_SHADER is validated automatically by AbstractProcessor::process(). Hence, there is finally no need anymore to validate these three different levels in each processor.

Openigtlink support

openigtlink processor implementation and matrix processor.
changes to mprrenderer and mprdemo are not relevant (should be in another branch)

Fixed various GCC and Cppcheck warnings.