Ticket #6822 (closed: fixed)
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: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: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
PG3_11490 is LaB6 collected at 10Hz.