GitLab

Depending on StringProperty's display type, its property widget shows either just the QLineEdit, or additionally a browse button that pops up a QFileDialog for either loading or saving a file, or choosing a directory.

closes #43

refs #474
refs #553

Statically linked builds strip all unused objects from their sources. This would be the case for the converters since they are nowhere called explicitly. In order to fix that, we use the same pattern as with the pipeline registration: The CMake build scripts parse all headers for an explicit template instantiation of a ConversionFunctionRegistrar. All these headers are included from the generated gen_converterregistration.h file, which is itself included from imagerepresentationconverter.cpp and thus gets compiled into the final executable also on static linking.

refs #474
refs #553

Fixed GenericGeometryTransferFunction not setting the clear color correctly before rendering into TF texture.
This also fixes the broken IXPV pipeline (Issue #62) that got also cleaned with this commit.

closes #62

google test merge request feedback reviewed. addressed all necessary major and minor issues that were pointed out; and hopefully fixed my stupidity in LinearMapping::operator==

Removed all code fragments in core code that was ITK specific (conversion from/to GenericImageRepresentationItk). The functionality was replaced by the new conversion functors in imagerepresentationconversionitk.h and imagerepresentationconversionitk.cpp.

refs #553
refs #474

Conversions between image representations are now managed at one central place: The ImageRepresentationConverter singleton uses the proven and established registration through static template instantiation idiom to register conversion functors during static initialization. Therefore, the ConversionFunctionRegistrar registers a conversion functor to a target representation type.

As proof-of-concept implementation, the former conversion API through T::tryConvertFrom, where T is a specific image representation, has been converted to the new API and merged into imagerepresentationconversioncore.h providing a conversion functor for each campvis-core representation.

Furthermore, implemented conversion from ImageRepresentationGL to GenericImageRepresentationLocal<>.

refs #553
refs #474

rewritten init() with the glcontextmanager instead of qtcontextmanager, updated ImageData, NumericProperty, GenericProperty test codes

added the documentation (brief). added test class for StringUtils. trimmed out the redundant code segments.

Removed ProcessorDecoratorShading and replaced all of its usages with the new LightSourceData API. This also includes adding LightSourceProviders to the corresponding pipelines.

refs #547

* Introducing base module with LightSourceProvider processor
* Updated GeometryRenderer and GeometryRendererDemo to use LightSourceData instead of ProcessorDecoratorShading
* Introducing LightSourceData::bind()

refs #547

refs #547

To now, each property hat an _invalidationLevel field that was evaluated by processors when the property had changed in order to determine what has to be done. However, since properties could also be owned by other classes, this design was semantically misleading.
Therefore, it was removed with this commit and replaced by the invalidation map of each processor. Now, this per-processor mapping of property -> invalidation level is managed by the processor itself. Furthermore, the invalidation level is no longer setup during property creation but during AbstractProcessor::addProperty(), which also makes much more sense.

ATTENTION: Due to these intrusive API changes, the code of all processors and other classes handling properties needs to be changed. As a reminder, the implementation of addProperty() also now takes a reference instead of a pointer, so that old code does no longer compile.

refs #542

* Introducing sigslot::signal::has_connections() to check whether a signal has listeners
* Fixed default OpenGL texture wrapping

Conflicts:
	modules/pipelinefactory.h

* Introduced tgt::Shader::IgnoreUniformLocationErrorGuard
* Fixed SimpleRaycaster::updateProperties() and OptimizedRaycaster::updateProperties() not calling base class' updateProperties()

* Updated ProcessorDecoratorGradient to support controllable LOD for gradient computation
* updated RaycastingProcessor to compute min/max depth values of entry/exit points and set as uniforms if needed by shader

* Introducing virtual GeometryData::hasTextureCoordinates()
* Introducing virtual GeometryData::applyTransformationToVertices()

Implemented normalizing conversion between different GenericImageRepresentationLocal<BASETYPE, NUMCHANNELS> representations as proof-of-concept. This conversion might not always yield expected results since it remaps the entire input data type range to the entire output data type range. While this is reasonable for downcasts (narrowing conversions) a histogram stretch for widening conversions is probably not expected or wanted by the caller.

A more flexible and elaborate conversion as well as automatic conversion from ITK and OpenGL representations will need a redesign of the conversion interface since we're fighting with severe cyclic include dependency problems here. The future design should provide the possibility to externally register conversion functions for new data types that are not necessarily known in the core package (i.e. ITK representations).

refs #474

Removed AbstractImage::getSubImage() interface since it's implementation in all subclasses is far from trivial and therefore was not done thoroughly. Since the functionality was not used anywhere so far, it was removed to avoid unexperienced users trying to use it and getting confused if it does not work as intended.
Could be reimplemented in extra processors for special cases.

closes #339

TGT and campvis-core used to manually define flags that instructed their
code to export DLL symbols. However, CMake has a property called
DEFINE_SYMBOL that can automatically define export flags when necessary.
Refactor the CMakeLists of TGT and campvis-core to make use of it.

References #367

CAMPVis used a custom option, CAMPVIS_SHARED_LIBS, to determine whether
shared or static libraries should be built. This option partially
duplicated the functionality of one of CMake's standard flags,
BUILD_SHARED_LIBS. The two could get out of sync, and because CMake only
consults BUILD_SHARED_LIBS when deciding how to build a library target,
it was possible to get inconsistent and broken build configurations.

To prevent that, add BUILD_SHARED_LIBS as an option to CAMPVis' main
CMakeLists and replace all references to CAMPVIS_SHARED_LIBS with
BUILD_SHARED_LIBS. This makes BUILD_SHARED_LIBS the only flag that
controls whether shared or static libraries are built.

References #367

Fixed TrackballNavigationEventListener setting the window ratio of its CameraProperty being directly overwritten due to concurrent access.

* Fixed duplicate symbols of template specialized TextFileParser::readAndParseItems<TextFileParser::ItemSeparatorLines> due to forced inline

refs #377

Builds and runs fine so far, but no elaborate testing done yet. Compiler throws a bunch of C4275 warning, since sigslot:has_slots<> is used in exported classes but is itself not exported. While this is no direct problem for campvis-internal builds, it should still be fixed some time.

refs #377