Ticket #4771 (closed: fixed)

Opened 9 years ago

Last modified 5 years ago

Instrument View: highlighting instrument components in the instrumentTreeWidget is very slow for TOPAZ.

Reported by: Janik Zikovsky Owned by: Janik Zikovsky
Priority: major Milestone: Release 2.1
Component: Mantid Keywords:
Cc: Blocked By:
Blocking: Tester: Dennis Mikkelson

Description

Open the instrument view on a TOPAZ workspace, go to the instrument tree, select a component to zoom into. It takes several seconds (~10 sec) to finally zoom in to the selected component.

Improve this performance.

Change History

comment:1 Changed 9 years ago by Janik Zikovsky

  • Status changed from new to accepted

comment:2 Changed 9 years ago by Janik Zikovsky

  • Takes 1.6 seconds in 3D view
  • In unwrapped view: takes 2.5 seconds the first time (caching assemblies)
    • Following times, it takes 1.7 seconds and does not zoom in correctly in unwrapped view
Last edited 9 years ago by Janik Zikovsky (previous) (diff)

comment:3 Changed 9 years ago by Janik Zikovsky

Refs #4771: highlighting a panel in instrument view much quicker

1.6 seconds->unmeasurably fast.

Changeset: 0e664d1c79308a04c9bef10a83b2a5aa25acd431

comment:4 Changed 9 years ago by Janik Zikovsky

Refs #4771 speed up of component picking

Changeset: 1f198c0765b9e13f3116369ff1eb68f3ca7b96a7

comment:5 Changed 9 years ago by Janik Zikovsky

Refs #4771: unwrapped view looks better when zoomed in

Changeset: 84cda6c311a6766dd87d110a4b08a48e5891d0bb

comment:6 Changed 9 years ago by Janik Zikovsky

Refs #4771: using bare component instead of parametrized for speed

6 seconds -> 1 second to calculate the assemblies size in unwrapped view.

Changeset: 30edbca9c568b5efb015113e29decffe6a0a2cf3

comment:7 Changed 9 years ago by Janik Zikovsky

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

It now takes ~ 1 second to zoom into a component the first time, and < 0.3 seconds each time thereafter, so it is 10-30 x faster than before.

comment:8 Changed 8 years ago by Dennis Mikkelson

  • Status changed from verify to verifying
  • Tester set to Dennis Mikkelson

comment:9 Changed 8 years ago by Dennis Mikkelson

  • Status changed from verifying to closed

Works fine on Snow Leopard and Linux. A bit slower on Windows 7, 64 bit, in particular if the Render mode is 'Cylindrical Y'.

comment:10 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​5618

Note: See TracTickets for help on using tickets.