Ticket #1589 (closed: fixed)

Opened 10 years ago

Last modified 5 years ago

EventWorkspace filtering by start/stop time

Reported by: Peter Peterson Owned by: Janik Zikovsky
Priority: major Milestone: Iteration 25
Component: Mantid Keywords:
Cc: Blocked By:
Blocking: Tester: Michael Whitty

Description

Filter event lists based on wall-clock times. There needs to be a design document discussing how this might work first.

Change History

comment:1 Changed 10 years ago by Janik Zikovsky

  • Owner set to Janik Zikovsky
  • Status changed from new to accepted
  • Component set to Mantid
  • Specify a start and end time.
    • Perhaps two ways: absolute time in ISO8601? format
    • And seconds since start, as a double.
  • Output: a new workspace, or replace the existing workspace. In both cases, a new EW will be created and filled.

comment:2 Changed 10 years ago by Janik Zikovsky

(In [6215]) Refs #1589: Added a filtering function for EventList and a FilterByTime algorithm to filter an EventWorkspace. Needs more tests.

comment:3 Changed 10 years ago by Janik Zikovsky

(In [6216]) Refs #1589: Fix windows test build.

comment:4 Changed 10 years ago by Janik Zikovsky

(In [6236]) Refs #1589: Event filtering by start/stop time; can be specified as relative or absolute times; proton charge and logs are also filtered by the same times.

comment:5 Changed 10 years ago by Janik Zikovsky

(In [6240]) Refs #1589: Fix Windows test build.

comment:6 Changed 10 years ago by Janik Zikovsky

(In [6244]) Refs #1589: Another try at fixing the other test builds. Also a little more docs.

comment:7 Changed 10 years ago by Janik Zikovsky

  • Summary changed from EventWorkspace filtering to EventWorkspace filtering by start/stop time

comment:8 Changed 10 years ago by Janik Zikovsky

(In [6245]) Refs #1589: And a 3rd time for Windows build. What is the proper way to do an absolute number of an integer under Windows?

comment:9 Changed 10 years ago by Janik Zikovsky

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

comment:10 Changed 10 years ago by Michael Whitty

  • Status changed from verify to verifying
  • Tester set to Michael Whitty

comment:11 Changed 10 years ago by Michael Whitty

  • Status changed from verifying to closed

Seems to work. Possibly some range checking on times input for future improvement, but I don't know if that's something the users would find helpful.

comment:12 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​2436

Note: See TracTickets for help on using tickets.