Ticket #5041 (closed: fixed)

Opened 9 years ago

Last modified 5 years ago

Count the number of events along pulse time

Reported by: Wenduo Zhou Owned by: Wenduo Zhou
Priority: major Milestone: Release 2.1
Component: Mantid Keywords:
Cc: petersonpf@… Blocked By:
Blocking: Tester: Jose Borreguero

Description

Count the number of events per pulse or a few pulses. The input is an EventWorkspace object. And the output can be either a Workspace2D object or an EventWorkspace object. Workspace2D is more straightforward, while EventWorkspace saves memory if run time is very long and flux is relatively low.

Change History

comment:1 Changed 9 years ago by Wenduo Zhou

Implement CountEventsInPulses. Refs #5041.

  1. Implement the algorithm CountEventInPulses.
  2. Add two more OPENMP definitions in MultiThreaded.h Changeset: e10b52dbc1aaec2cef71c68a9f37dd51a39f6892

comment:2 Changed 9 years ago by Wenduo Zhou

Update CMakeList. Refs #5041.

Changeset: e58219f9bd43a647d60d682b37d774fbca588fd4

comment:3 Changed 9 years ago by Wenduo Zhou

Add empty unit test. Refs #5041.

Changeset: 5b099bd1a30fc23e60bbdee00e3c8cc29e8e0264

comment:4 Changed 9 years ago by Wenduo Zhou

  • Status changed from new to accepted

comment:5 Changed 9 years ago by Wenduo Zhou

Make Windows happy. Refs #5041.

Changeset: 9e4f9acd3e41a0ed9a38d7cdc1230cf01fab416a

comment:6 Changed 9 years ago by Wenduo Zhou

Add unit test. Refs #5041.

Changeset: 4456b916348866eb78460a83d0f583b86e8340fd

comment:7 Changed 9 years ago by Russell Taylor

Re #5041. Disable openmp usage for now - it can make unit test slow

...when running alongside other unit tests (i.e. with ctest -j). It sometimes hangs until all other unit tests have finished.

Changeset: 006cf72bc233ccf04cc255ed1e297836aaf8c21b

comment:8 Changed 9 years ago by Wenduo Zhou

Modification on calling OpenMP. Refs #5041.

Add a new option "parallel" to the algorithm. Turn off the parallelization in unit test to make the test time shorter.

Correct the name of some difinition for OpenMP in MultiThreaded.h

Changeset: 0a86394c29c5fd944bf8cc5ae1be66564a70dd10

comment:9 Changed 8 years ago by Wenduo Zhou

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

comment:10 Changed 8 years ago by Jose Borreguero

  • Status changed from verify to verifying
  • Tester set to Jose Borreguero

comment:11 Changed 8 years ago by Jose Borreguero

  • Status changed from verifying to closed

comment:12 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​5887

Note: See TracTickets for help on using tickets.