Ticket #1722 (closed: fixed)

Opened 10 years ago

Last modified 5 years ago

3D instrument view: Plot RectangularDetectors as bitmaps rather than individual pixels

Reported by: Janik Zikovsky Owned by: Janik Zikovsky
Priority: major Milestone: Iteration 25
Component: Mantid Keywords:
Cc: Blocked By:
Blocking: Tester: Vickie Lynch

Description

... this should greatly speed up instrument rendering.

Change History

comment:1 Changed 10 years ago by Janik Zikovsky

(In [6743]) Refs #1722: Added a BitmapGeometryRenderer to display RectangularDetector objects as bitmaps instead of millions of cubes. Added support in the instrument widget for it. Modified SNAP definition to use the RectangularDetector type of detector. Remaining problems: bounding boxes; and the data is not being shown on RectangularDetectors in the 3D view, not yet.

comment:2 Changed 10 years ago by Janik Zikovsky

  • Status changed from new to accepted

comment:3 Changed 10 years ago by Janik Zikovsky

(In [6746]) Refs #1722: Fixed qtiplot.pro

comment:4 Changed 10 years ago by Janik Zikovsky

(In [6748]) Refs #1722: Reverted qtiplot.pro

comment:5 Changed 10 years ago by Janik Zikovsky

(In [6823]) Refs #1722: RectangularDetector's now display as bitmaps in the 3D instrument view. BoundingBox issue fixed, but the code of BoundingBoxes is a little messy in some parts of the code and could use cleaning up. SNAP geometry updated to use RectangularDetector tag. Added the BitmapGeometryHandler to render the bitmaps (in the Geometry project). The RectangularDetectorActor is in qtiplot to handle those required calculation.

comment:6 Changed 10 years ago by Janik Zikovsky

(In [6825]) Refs #1722: Update to SNAP instrument (removed duplicate attribute) and mini change to trigger a rebuild of QtiPlot - it had built against an old version of the framework and therefore failed, because the qtiplot build does not wait for the framework build to complete.

comment:7 Changed 10 years ago by Janik Zikovsky

(In [6856]) Refs #1722: RectangularDetector's now show up in the instrument tree with all their sub-pixels.

comment:8 Changed 10 years ago by Janik Zikovsky

(In [6946]) Refs #1749 and #1722: Added IRectangularDetector and ParRectangularDetector in order to handle parametrized locations. I seem to be having trouble getting the values from the DAS logs, but the plotting works though.

comment:9 Changed 10 years ago by Janik Zikovsky

(In [6947]) Refs #1722: Touching qtiplot in order to force a rebuild, because on the build server, qtiplot builds before the framework, which is a little silly.

comment:10 Changed 10 years ago by Janik Zikovsky

  • Status changed from accepted to verify
  • Resolution set to fixed

Will hopefully be refactored because it is kind of ugly, but it works!

comment:11 Changed 10 years ago by Vickie Lynch

  • Status changed from verify to verifying
  • Tester set to Vickie Lynch

comment:12 Changed 10 years ago by Vickie Lynch

  • Status changed from verifying to closed

Works for SNAP and PG3 data

comment:13 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​2569

Note: See TracTickets for help on using tickets.