Ticket #5618 (closed: fixed)

Opened 8 years ago

Last modified 5 years ago

Add a FilteredTimeSeriesProperty type

Reported by: Martyn Gigg Owned by: Martyn Gigg
Priority: critical Milestone: Release 2.2
Component: Mantid Keywords:
Cc: Blocked By:
Blocking: Tester: Owen Arnold

Description

This will inherit from TimeSeriesProperty but will allow access back to the original unfiltered data.

Change History

comment:1 Changed 8 years ago by Martyn Gigg

  • Status changed from new to accepted

comment:2 Changed 8 years ago by Martyn Gigg

Move TimeSeriesProperty impl to cpp. Refs #5618

This saves the entire definition being pulled into every file using it & makes it much easier to change.

Changeset: 3ccfefad3bae7e3ef23042c3703e73dffe2d28ba

comment:3 Changed 8 years ago by Martyn Gigg

Fix missing concrete instantiation for statistics. Refs #5618

Changeset: c8f1de761fb5ddc9f3bbdad5ad0d864666f3f6fc

comment:4 Changed 8 years ago by Martyn Gigg

Further property instantiations. Refs #5618

Changeset: fd04c5892295d555a28b6a8d1fdeee8b668d64cd

comment:5 Changed 8 years ago by Martyn Gigg

Add a FilteredTimeSeriesProperty. Refs #5618

Changeset: b333babd3208bf679345d4038e715603c82f4eda

comment:6 Changed 8 years ago by Martyn Gigg

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

comment:7 Changed 8 years ago by Martyn Gigg

Better control over ownership in FilterTimeSeriesProperty. Refs #5618

Changeset: dab6cf8fdae6e0b184bb32d487f237e4d7a66d8e

comment:8 Changed 8 years ago by Martyn Gigg

Allow LogFilter to simply combine filters. Refs #5618

Changeset: 2631366080245c5c72f173bfa8a9527704169125

comment:9 Changed 8 years ago by Martyn Gigg

Fix MantidPlot build. Refs #5618

Changeset: e3e6deb57168efaeefd031fdf83ce86d72c9d733

comment:10 Changed 8 years ago by Martyn Gigg

Export FilteredTimeSeriesProperty to Python. Refs #5618

If it is not exported then any property returned by getRun will be given as the Property type.

Changeset: c018b897e69e741582cd4f93cf2a308c54e3c1bf

comment:11 Changed 8 years ago by Martyn Gigg

Move TimeSeriesProperty impl to cpp. Refs #5618

This saves the entire definition being pulled into every file using it & makes it much easier to change.

Changeset: 3ccfefad3bae7e3ef23042c3703e73dffe2d28ba

comment:12 Changed 8 years ago by Martyn Gigg

Fix missing concrete instantiation for statistics. Refs #5618

Changeset: c8f1de761fb5ddc9f3bbdad5ad0d864666f3f6fc

comment:13 Changed 8 years ago by Martyn Gigg

Further property instantiations. Refs #5618

Changeset: fd04c5892295d555a28b6a8d1fdeee8b668d64cd

comment:14 Changed 8 years ago by Martyn Gigg

Add a FilteredTimeSeriesProperty. Refs #5618

Changeset: b333babd3208bf679345d4038e715603c82f4eda

comment:15 Changed 8 years ago by Owen Arnold

  • Status changed from verify to verifying
  • Tester set to Owen Arnold

comment:16 Changed 8 years ago by Owen Arnold

  • Status changed from verifying to closed

Code changes look sensible. Running & passing unit tests provide coverage for this new functionality.

comment:17 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 6464

Note: See TracTickets for help on using tickets.