Ticket #1613 (closed: fixed)

Opened 10 years ago

Last modified 5 years ago

Intraument View: Plot entire tubes rather than seperate pixels

Reported by: Nick Draper Owned by: Roman Tolchenov
Priority: critical Milestone: Iteration 25
Component: Mantid Keywords:
Cc: Blocked By:
Blocking: Tester: Martyn Gigg

Description (last modified by Nick Draper) (diff)

  1. define the object shape of ComponentAssemblies (perhaps as PlottableComponentAssemblies?)
  2. When plotting add these to the list of plottable object rather than the constituent pixels
  3. When colouring create a texture map of the colours and wrap it around

A good test of this is if it can cope with the SetScalingPSD algorithm.

Change History

comment:1 Changed 10 years ago by Nick Draper

  • Summary changed from Plot entire tubes (objectComponents) rather than seperate pixels to Intraument View: Plot entire tubes rather than seperate pixels

comment:2 Changed 10 years ago by Nick Draper

  • Description modified (diff)

comment:3 Changed 10 years ago by Roman Tolchenov

  • Status changed from new to accepted
  • Component set to Mantid

comment:4 Changed 10 years ago by Roman Tolchenov

(In [7029]) Added ObjCompAssembly class to represent an assembly that can draw its overall shape. re #1613

comment:5 Changed 10 years ago by Roman Tolchenov

(In [7031]) Forgot to add a few files... re #1613

comment:6 Changed 10 years ago by Roman Tolchenov

(In [7071]) Added texturing to ObjCompAssembly. re #1613

comment:7 Changed 10 years ago by Roman Tolchenov

(In [7073]) Fixing the build. re #1613

comment:8 Changed 10 years ago by Roman Tolchenov

(In [7076]) Debug printout deleted. re #1613

comment:9 Changed 10 years ago by Roman Tolchenov

(In [7078]) Removed unused GLU function breaking the linux build. re #1613

comment:10 Changed 10 years ago by Roman Tolchenov

(In [7142]) Made cuboid ObjCompAssemblies work. re #1613

comment:11 Changed 10 years ago by Roman Tolchenov

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

comment:12 Changed 10 years ago by Martyn Gigg

  • Status changed from verify to verifying
  • Tester set to Martyn Gigg

comment:13 Changed 10 years ago by Martyn Gigg

  • Status changed from verifying to reopened
  • Resolution fixed deleted

comment:14 Changed 10 years ago by Roman Tolchenov

(In [7148]) Removed the lighting button. re #1613

comment:15 Changed 10 years ago by Roman Tolchenov

(In [7151]) Corrected ObjCompAssemblyActor to use parametrised instrument properly. re #1613

comment:16 Changed 10 years ago by Roman Tolchenov

(In [7165]) A fix to a problem (at least on my machine) of drawing the first frame of the instrument view. re #1613

comment:17 Changed 10 years ago by Roman Tolchenov

(In [7168]) Modified MERLIN instrument definition file for testing ObjCompAssembly. re #1613

comment:18 Changed 10 years ago by Roman Tolchenov

(In [7169]) The first frame issue wasn't actually fixed, but it is fixed now. re #1613

comment:19 Changed 10 years ago by Roman Tolchenov

  • Status changed from reopened to accepted

comment:20 Changed 10 years ago by Roman Tolchenov

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

comment:21 Changed 10 years ago by Martyn Gigg

  • Status changed from verify to verifying

comment:22 Changed 10 years ago by Martyn Gigg

Tested on Windows, Linux and Mac with MERLIN and speed improvements are mighty impressive.

Should consider activating this for the other compatible instruments, WISH etc?

comment:23 Changed 10 years ago by Martyn Gigg

  • Status changed from verifying to closed

SetScalingPSD portion splintered off into Re #1831.

Closing this ticket given the new one.

comment:24 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​2460

Note: See TracTickets for help on using tickets.