Ticket #6822 (closed: fixed)

Opened 8 years ago

Last modified 5 years ago

RemovePromptPulse should remove all prompt pulses

Reported by: Peter Peterson Owned by: Peter Peterson
Priority: major Milestone: Release 2.5
Component: Mantid Keywords:
Cc: Blocked By:
Blocking: Tester: Wenduo Zhou

Description

Since some instruments run at lower frequencies that the accellerator (i.e. 30Hz on a 60Hz sorce), modify RemovePromptPulse to remove all occurences of the prompt pulse in the data.

Change History

comment:1 Changed 8 years ago by Peter Peterson

  • Status changed from new to accepted

PG3_11490 is LaB6 collected at 10Hz.

comment:2 Changed 8 years ago by Peter Peterson

Re #6822. Added support for multiple prompt pulses.

Changeset: e87c9c4a468bef7e3c68b8035898c5b3bd6e6762

comment:3 Changed 8 years ago by Peter Peterson

Re #6822. Added support for multiple prompt pulses.

Changeset: e87c9c4a468bef7e3c68b8035898c5b3bd6e6762

comment:4 Changed 8 years ago by Peter Peterson

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

This was done in ​feature/6822_RemovePromptPulse.

To test it try out data at 60Hz (done by the system test) as well as at a lower frequecy. PG3_11490 is a little hard to see the removed data past the third prompt pulse because the statistics aren't so good.

comment:5 Changed 8 years ago by Wenduo Zhou

  • Status changed from verify to verifying
  • Tester set to Wenduo Zhou

comment:6 Changed 8 years ago by Wenduo Zhou

  • Status changed from verifying to closed

comment:7 Changed 8 years ago by Peter Peterson

Re #6822. Added support for multiple prompt pulses.

Changeset: b381b7a65606ae0953a46537ed631663e6213375

comment:8 Changed 8 years ago by Peter Peterson

Re #6822. Added support for multiple prompt pulses.

Changeset: b381b7a65606ae0953a46537ed631663e6213375

comment:9 Changed 7 years ago by Peter Peterson

Re #6822. Added support for multiple prompt pulses.

Changeset: b381b7a65606ae0953a46537ed631663e6213375

comment:10 Changed 7 years ago by Peter Peterson

Re #6822. Added support for multiple prompt pulses.

Changeset: b381b7a65606ae0953a46537ed631663e6213375

comment:11 Changed 7 years ago by Peter Peterson

Re #6822. Added support for multiple prompt pulses.

Changeset: b381b7a65606ae0953a46537ed631663e6213375

comment:12 Changed 7 years ago by Peter Peterson

Re #6822. Added support for multiple prompt pulses.

Changeset: b381b7a65606ae0953a46537ed631663e6213375

comment:13 Changed 7 years ago by Peter Peterson

Re #6822. Added support for multiple prompt pulses.

Changeset: b381b7a65606ae0953a46537ed631663e6213375

comment:14 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​7668

Note: See TracTickets for help on using tickets.