Ticket #9436 (closed: fixed)
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: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