Page MenuHomePhabricator
Create Task
Maniphest T9815

Warnings/errors when opening specific perspectives
Closed, ResolvedPublic
Actions

Description

C) Warnings/errors when opening specific perspectives

C1) Open IVIM Perspective

QLayout: Attempting to add QLayout "" to QWidget "m_VisualizeResultsWidget", which already has a layout

C2) Open TBSS Perspective

WARNING: Trying to create an executable extension from a non-CTK plug-in. Use CreateExecutableExtension<C>(...) method instead.

C3) Open Connectomics Perspective

WARNING: Trying to create an executable extension from a non-CTK plug-in. Use CreateExecutableExtension<C>(...) method instead.

C4) Open Screenshots Movies Perspective

No such slot QmitkScreenshotMaker::BlockControls()
No such slot QmitkScreenshotMaker::UnBlockControls()
No such slot QmitkScreenshotMaker::UnBlockControlsMovieDeactivate()

Related Objects

Event Timeline

kislinsk added projects: MITK, Bugzilla.Oct 19 2011, 2:48 PM
• hering added a subscriber: • hering.Oct 20 2011, 2:11 PM

The cause of the problems described in (C4) are missing slots. Since de/activating is not relevant for a ScreenShot Maker, also the signals and the connections will be removed. Peter is now creating a new design for the ScreenshotMaker, it will be fixed there to avoid merge-conflicts.

• hering added a comment.Oct 20 2011, 3:51 PM

(C2) and (C3) occur because both TBSS and Connectomics are MBI-Plugins. The nice solution is to make both of them CTK-conform plugins. The faster is to deactivate the Berry Warning thrown.

• hering added a comment.Oct 20 2011, 4:01 PM

The application outputs an error message

ERROR: File does not exist, or cannot be read. Filename =
[7.990] Could not open file!

when opening any of the views throuhg the WelcomeScreen. This happens because of QmitkDiffusionImagingAppIntroPart.cpp is trying to open a file when a perspective is activated (it is a code reminder from AtlasApp ). I have removed the code (see the remote topic branch)

Klaus, could you please check whether there are more code lines which are not needed by the DI-App?

fritzsck added a subscriber: fritzsck.Oct 27 2011, 8:09 PM

we should be fine I think..

• goch added a subscriber: • goch.Oct 28 2011, 11:00 AM

Has been merged into the release, probably should be merged into the master as well, removing the block

• hering added a comment.Nov 10 2011, 11:56 AM

[6c8bfa]: Merge branch 'bug-9815-DIApp-Intro-Warnings'

Merged commits:

2011-10-20 16:02:56 Jan Hering [750b8b]
Removed automatic data loading for DIAppIntroPart

  • this was a functionality needed by the Atlas-App, since the activation

of a perspective automatically invoked the loading of a corresponding
fiber data.

• hering added a comment.Nov 10 2011, 2:09 PM

Relevant changes merged into master. Closing bug.

kislinsk mentioned this in T27033: Clean up stale remote branches.Jan 25 2020, 9:01 PM
MITK · MIC · IMSY · DKFZ