Ticket #5985 (closed: fixed)

Opened 8 years ago

Last modified 5 years ago

Speed up RebinByPulseTimes

Reported by: Owen Arnold Owned by: Owen Arnold
Priority: major Milestone: Release 2.3
Component: Mantid Keywords:
Cc: Blocked By: #5948
Blocking: Tester: Nick Draper

Description

This algorithm is currently running single threaded. It should be possible to improve the performance of this algorithm in multi-core environments. Since I have already put performance tests in place, I should be able to improve the speed against the current benchmarks.

Attachments

RebinByPulseTimePerformance.png​ (287.9 KB) - added by Owen Arnold 8 years ago.

Change History

comment:1 Changed 8 years ago by Owen Arnold

  • Status changed from new to accepted

comment:2 Changed 8 years ago by Owen Arnold

refs #5985. Process in parallel.

Changeset: 9880fcae01f767e61909f88e97894a499e74a4ee

Changed 8 years ago by Owen Arnold

comment:3 Changed 8 years ago by Owen Arnold

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

Tester: The old version wasn't running long enough to build up performance statistics, but as the attached graph generated. The parallelisation has resulted in a speed increase of this algorithm.

comment:4 Changed 8 years ago by Nick Draper

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

comment:5 Changed 8 years ago by Nick Draper

  • Status changed from verifying to closed

tested as part of #5948, speed acceptable

comment:6 Changed 8 years ago by Owen Arnold

refs #5985. Process in parallel.

Changeset: 9880fcae01f767e61909f88e97894a499e74a4ee

comment:7 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​6831

Note: See TracTickets for help on using tickets.