Ticket #4486 (closed: fixed)

Opened 9 years ago

Last modified 5 years ago

Instrument View: Unwrapped view is quite 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 (last modified by Janik Zikovsky) (diff)

Pretty much unusable!

Change History

comment:1 Changed 9 years ago by Janik Zikovsky

  • Priority changed from minor to major
  • Description modified (diff)

comment:2 Changed 9 years ago by Janik Zikovsky

  • Summary changed from Instrument Unwrapped view is quite slow for TOPAZ. to Instrument View: Unwrapped view is quite slow for TOPAZ.

comment:3 Changed 9 years ago by Janik Zikovsky

  • Status changed from new to accepted

comment:4 Changed 9 years ago by Janik Zikovsky

Starting point: it currently takes 21 seconds to open the unwrapped view.

comment:5 Changed 9 years ago by Janik Zikovsky

Refs #4486: TOPAZ unwrapped view takes about 10 seconds

Unfortunately could not parallellize because Detector->getPos() is not quite thread safe

Changeset: 6a752ee4ca1f9e41e98e5c4fdc98b752e89b7c39

comment:6 Changed 9 years ago by Janik Zikovsky

Refs #4486: speed up unwrapped view by parallelizing it

It takes about 10 seconds the first time, then 2 seconds each other time. This compares to 21 seconds each time before

Changeset: 820f53a0cad7ad273c31afa85efd4130d28dacf2

comment:7 Changed 9 years ago by Janik Zikovsky

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

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 Linux and Mac, still slow on Windows 7. See ticket #5233 for next iteration.

comment:10 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​5333

Note: See TracTickets for help on using tickets.