Ticket #1396 (closed: wontfix)

Opened 10 years ago

Last modified 5 years ago

Attempt to parallelize LoadEventPreNexus

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

Description


Change History

comment:1 Changed 10 years ago by Janik Zikovsky

  • Status changed from new to accepted

comment:2 Changed 10 years ago by Janik Zikovsky

(In [5298]) Refs #1396: some code to work towards parallelized event loading, but does not work yet. Refs #1406: load an instrument geometry file. Test disabled because it does not pass yet.

comment:3 Changed 10 years ago by Janik Zikovsky

  • Milestone changed from Iteration 24 to Iteration 25

Pushing this to next iteration. May not be possible without pre-sorting. EventNexus files may help this problem.

comment:4 Changed 10 years ago by Janik Zikovsky

(In [5889]) Refs #1396: Added test to make sure struct data sizes are correct.

comment:5 Changed 10 years ago by Janik Zikovsky

(In [6124]) Refs #1396: Successfully parallelized the loadEventPreNexus algorithm; however, it only runs 10-15% faster, and that only on large files and with the right settings. Therefore I have disabled the parallelized loading option in this checkin, so that it does not clutter the UI needlessly.

comment:6 Changed 10 years ago by Janik Zikovsky

(In [6125]) Refs #1396: Test did not have the mapping file specified; for some reason the servers found it but my local install did not. to avoid this, the mapping file is now specified in the test.

comment:7 Changed 10 years ago by Janik Zikovsky

  • Status changed from accepted to verify
  • Resolution set to wontfix
  • Component set to Mantid

This feature probably will not work until EventNexus files. Will leave the code in for now, but disabled.

comment:8 Changed 10 years ago by Nick Draper

  • Status changed from verify to verifying
  • Tester set to Nick Draper

comment:9 Changed 10 years ago by Nick Draper

  • Status changed from verifying to closed

comment:10 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​2243

Note: See TracTickets for help on using tickets.