Page MenuHomePhabricator

Fix various reported little things
Closed, ResolvedPublic

Description

@maleike reported:

  • Image Navigator: zero padding between slider and spinbox to its right. That looks broken
  • BasicImageProcessing:
    • plugin title is in CamelCase, looks more like a variable than a readable title (Basic Image Processing)
    • Qt loayout is inconsistent with other plugins, there is 0 padding within the group boxes. That looks too “pressed”
  • Volume Visualization:
    • could not get to visualize any volume visualization, no logged errors/warnings. Image was non-human CT, transfer function was chosen to clearly include most of the data.
    • Crashed the application after some selection of different visualizations
  • Screenshot Maker:
    • Options panel with ‘x’, ‘y’, ‘z’: unclear meaning. Ok, somehow described in Help Page. Suggestion: add label “Set camera view direction:”
    • “Options” are only “Background Color”. The other buttons seem to be “Camera control”
    • Keep pressing ‘x’ to keep the camera turning around. Perhaps this is the intended behavior? In that case add a label “Rotate camera around axis:”
  • PointSet Interaction:
    • Title: is it PointSet or Point Set. Or even Point Cloud?
  • Image Cropper:
    • “File Name” is selected object name, no relation to a file. This label is pure black, hardly readable
    • Red labels are also hard to read (but ok if this is the default warning color around all plugins)
    • The logic seems too complicated without a reason, adding a single bounding box seems to suffice, could be added in Data Manager. Why choose a name for the object, I want to use it quickly. If I want to rename I can use Data Manager
  • Resampling:
    • Not enough padding in group boxes, controls are glued to their parent containers
    • Resampling crashes (1000x1000x1000 voxels), although RAM is largely sufficient
    • What is the idea of “Resample multiple images”?? There is no help page and it is not obvious at all. Is that like an image pyramid?! Is it just resampling multiple iamges selected in Data Manger (in that case I’d just expect the view to warn the user that resampling will apply to multiple images)
  • OverLay Manager:
    • Inconsistent menu entry in “View/Overlay Manager” and view title (“Properties (Annotations”)
    • Function can be discovered: good
    • Tree view for properties collapses entries when clicking a checkbox. This is very annoying
    • Tested only for ColorBarAnnotation: essential properties cannot be edited (margin, lookup table). Alignment does not seem to have any effect
    • Seems very buggy/experimental overall. High potential however :-)
  • Segmentation:
    • Allocation of segmentation seemed very slow for what it is (new uint8_t[number] in release mode)
    • Crash on first “Add tool” interaction (just selected CT image, “New…”, activate “Add”, attempt to draw one loop in axial view)
    • “Add” worked later on
    • 2D Fast Marching brings only warning logs on console regarding wrongly define interactor
    • Help Page: LiveWire section is missing an image
    • The tools tabs (2D Tools, 3D Tools) are designed as the opposite of other views: approx.. 2x the regular padding
    • Some/Most operations seems very slow, hardly interactive. This seems related to the display of 2D contours (hypothesis not verified)
    • Otsu is using a single core only