Ticket #4904 (closed: fixed)

Opened 9 years ago

Last modified 5 years ago

Refactor TimeSeriesProperty

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

Description

Replace the container in TimeSeriesProperty from multimap to vector in order to increase the speed. The API should not be altered.

Change History

comment:1 Changed 9 years ago by Wenduo Zhou

  • Status changed from new to accepted

comment:2 Changed 9 years ago by Wenduo Zhou

Refactor TimeSeriesProperty. Refs #4904.

Data container in TimeSeriesProperty is changed from multimpa to vector. All unit tests are passed.

Changeset: 53ff3880f84f764609948f19af21873a70086692

comment:3 Changed 9 years ago by Wenduo Zhou

Correct unit test. Refs #4904.

The previous clearFilter() has a defect due to saving the multimap to map to restore the original multimap.

Changeset: 7dd5a9417879e88368a3713d2a6d2bafd6fe9b6e

comment:4 Changed 9 years ago by Wenduo Zhou

Correct a previous error in test. Refs #4904.

Changeset: c86d1f97efc6058752e18cfdce9e621b17f91920

comment:5 Changed 9 years ago by Wenduo Zhou

Modify due to TimeSeriesProperty. Refs #4904.

A change in TimeSeriesProperty's API causes this modification. This change in API is trivial and only affects unit test.

Changeset: 0aa311eaadd6035d96969c44dec466cfa0d2b612

comment:6 Changed 9 years ago by Wenduo Zhou

Make change due to TimeSeriesProperty. Refs #4904.

TimeSeriresProperty has change in API for addValue(). It is trivial and only unit tests are affected.

Changeset: fdd9b3ad246465b12ac8ec6d77916c8cac5dd025

comment:7 Changed 9 years ago by Wenduo Zhou

Clean codes and fix a bug. Refs #4904.

Changeset: 1df2b608f984b6022ee7bd8c43e7526cd73d950d

comment:8 Changed 9 years ago by Wenduo Zhou

Correct unit test from a bug. Refs #4904.

Changeset: 53aa423d671f565f73bd22a2d328a3eb230e7459

comment:9 Changed 9 years ago by Wenduo Zhou

Refactor TimeSeriesProperty. Refs #4904.

Data container in TimeSeriesProperty is changed from multimpa to vector. All unit tests are passed.

Changeset: 53ff3880f84f764609948f19af21873a70086692

comment:10 Changed 9 years ago by Wenduo Zhou

Correct unit test. Refs #4904.

The previous clearFilter() has a defect due to saving the multimap to map to restore the original multimap.

Changeset: 7dd5a9417879e88368a3713d2a6d2bafd6fe9b6e

comment:11 Changed 9 years ago by Wenduo Zhou

Correct a previous error in test. Refs #4904.

Changeset: c86d1f97efc6058752e18cfdce9e621b17f91920

comment:12 Changed 9 years ago by Wenduo Zhou

Modify due to TimeSeriesProperty. Refs #4904.

A change in TimeSeriesProperty's API causes this modification. This change in API is trivial and only affects unit test.

Changeset: 0aa311eaadd6035d96969c44dec466cfa0d2b612

comment:13 Changed 9 years ago by Wenduo Zhou

Make change due to TimeSeriesProperty. Refs #4904.

TimeSeriresProperty has change in API for addValue(). It is trivial and only unit tests are affected.

Changeset: fdd9b3ad246465b12ac8ec6d77916c8cac5dd025

comment:14 Changed 9 years ago by Wenduo Zhou

Clean codes and fix a bug. Refs #4904.

Changeset: 1df2b608f984b6022ee7bd8c43e7526cd73d950d

comment:15 Changed 9 years ago by Wenduo Zhou

Correct unit test from a bug. Refs #4904.

Changeset: 53aa423d671f565f73bd22a2d328a3eb230e7459

comment:16 Changed 9 years ago by Wenduo Zhou

More refactoring on the codes. Refs #4904.

Changeset: 27b0898508615f744a08b252881a25a2d1821700

comment:17 Changed 9 years ago by Wenduo Zhou

Refactor codes. Refs #4904.

Using STL library in searching.

Changeset: 44b8dc9f9977fbc11a2764b2a31cb030b5ed9713

comment:18 Changed 9 years ago by Wenduo Zhou

Clean the codes. Refs #4904.

Changeset: ad791146a4b1d90b492371c646f5004066439281

comment:19 Changed 9 years ago by Wenduo Zhou

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

comment:20 Changed 8 years ago by Roman Tolchenov

  • Status changed from verify to verifying
  • Tester set to Roman Tolchenov

comment:21 Changed 8 years ago by Roman Tolchenov

  • Status changed from verifying to closed

comment:22 Changed 8 years ago by Wenduo Zhou

Fix Log filter. Refs #4904.

Changeset: 1754bb3bbb977ded69bdefe4c98579aaba08cec6

comment:23 Changed 8 years ago by Wenduo Zhou

Fix Log filter. Refs #4904.

Changeset: 1754bb3bbb977ded69bdefe4c98579aaba08cec6

comment:24 Changed 8 years ago by Wenduo Zhou

Fix some unit tests. Refs #4904.

Changeset: 1a1499f9ea4c034e0e5e7ef84b28f6c23f10d3ad

comment:25 Changed 8 years ago by Wenduo Zhou

Remove a warning. Refs #4904.

Changeset: b5d6c8ff26e1be0cab680f354b83d6929c10dce2

comment:26 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​5750

Note: See TracTickets for help on using tickets.