Ticket #9436 (closed: fixed)

Opened 6 years ago

Last modified 5 years ago

Algorithm that merges POLDI data after performing checks

Reported by: Michael Wedel Owned by: Michael Wedel
Priority: major Milestone: Release 3.2
Component: Diffraction Keywords:
Cc: Blocked By:
Blocking: Tester: Owen Arnold

Description

Sometimes experimental data from POLDI runs can be merged. The count data from several hdf-files are simply added, which can be achieved using the Plus-algorithm. But before, some sanity-checks have to be performed, since it is only allowed to merge data from experiments with the exact same parameters (sample table position, chopper speed, ...).

Change History

comment:1 Changed 6 years ago by Michael Wedel

  • Component changed from Framework to Diffraction

comment:2 Changed 6 years ago by Michael Wedel

  • Milestone changed from Backlog to Release 3.2

comment:3 Changed 6 years ago by Nick Draper

  • Status changed from new to assigned

comment:4 Changed 6 years ago by Michael Wedel

  • Status changed from assigned to inprogress

Refs #9436. Pulling in changes for POLDI merge algorithm

Changeset: 10eb66b029655b37e954293f9130a2699ac54d4e

comment:5 Changed 6 years ago by Michael Wedel

Refs #9436. Pulling in changes for POLDI merge algorithm

Changeset: 10eb66b029655b37e954293f9130a2699ac54d4e

comment:6 Changed 6 years ago by Michael Wedel

Refs #9436. Adding merge algorithm for POLDI data

Changeset: d784ce4b1ea26dcee48b9bcacbf17ee8b089dcb4

comment:7 Changed 6 years ago by Michael Wedel

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

Algorithm only merges POLDI data when it is sensible to do so.

Testing information: Unit test is in place, system test is available as well to ensure correct behavior.

comment:8 Changed 6 years ago by Owen Arnold

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

comment:9 Changed 6 years ago by Owen Arnold

  • Status changed from verifying to closed

Merge remote-tracking branch 'origin/feature/9436_poldi_data_file_merging_algorithm'

Full changeset: c5e97214ae2165eb5afcc218f8a93457e566ee39

comment:10 Changed 6 years ago by Owen Arnold

Fine work.

May I suggest that you (as part of a separate minor ticket) expand the wiki description to include information on exactly what checks are being performed.

comment:11 Changed 6 years ago by Michael Wedel

Refs #9436. Make test compatible with changes from #9564.

Changeset: ca35cf043aa6c1f6d934c01136a4e10094389055

comment:12 Changed 6 years ago by Michael Wedel

Merge branch 'feature/9436_poldi_merge_system_test' into feature/10074_merge_poldi_system_tests

Full changeset: 33f206743d4f2e93cc829b61b40fd9c6804044a5

comment:13 Changed 6 years ago by Martyn Gigg

Merge branch 'feature/9436_poldi_merge_system_test'

Full changeset: d58d5e9494ba074d0c1c7c0111da964323fb0c3c

comment:14 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 10279

Note: See TracTickets for help on using tickets.