Ticket #5587 (closed: fixed)

Opened 8 years ago

Last modified 5 years ago

Improve algorithms that set a conventional cell and TransformHKL

Reported by: Dennis Mikkelson Owned by: Dennis Mikkelson
Priority: major Milestone: Release 2.2
Component: Mantid Keywords:
Cc: petersonpf@…, lynchve@…, mikkelsonr@…, pascal.manuel@…, nick.draper@… Blocked By:
Blocking: Tester: Vickie Lynch

Description

If peaks in a peaks workspace have been successfully indexed, changing the cell, should NOT change which peaks are indexed. However, currently TransformHKL and the algorithms that set a conventional cell all transform the UB matrix and then re-index the peaks with the new UB. As a result, some peaks that were previously indexed may not meet the hkl tolerance for the new UB, and so might not be indexed. If the cell is changed, it would be preferable to just transform the existing indices and the UB, without actually re-indexing the peaks.

Change History

comment:1 Changed 8 years ago by Dennis Mikkelson

RE: 5587, Improve Conv. Cell algorithms & TransformHKL

The algorithms: SelectCellOfType(), SelectCellWithForm() and TransformHKL() now apply a corresponding transformation to previously computed HKLs, rather than completely reindexing all peaks with the new UB. When changing to a larger conventional cell, this typically gets more peaks correctly indexed. If the previous behaviour is required, the IndexPeaks() algorithm can be applied separately to the peaks workspace, after running SelectCellOfType(), SelectCellWithForm() or TransformHKL() refs #5587

Changeset: 1ea62732d0f601b74f1e3dcaa404d737df7edb09

comment:2 Changed 8 years ago by Dennis Mikkelson

RE: 5587, Fix problem with compile under VC

refs #5587

Changeset: e29047fbe0e6ea860229716d8e829c4aae8fc1b9

comment:3 Changed 8 years ago by Dennis Mikkelson

  • Status changed from new to accepted

comment:4 Changed 8 years ago by Dennis Mikkelson

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

comment:5 Changed 8 years ago by Dennis Mikkelson

RE: 5587, Improve Conv. Cell algorithms & TransformHKL

The algorithms: SelectCellOfType(), SelectCellWithForm() and TransformHKL() now apply a corresponding transformation to previously computed HKLs, rather than completely reindexing all peaks with the new UB. When changing to a larger conventional cell, this typically gets more peaks correctly indexed. If the previous behaviour is required, the IndexPeaks() algorithm can be applied separately to the peaks workspace, after running SelectCellOfType(), SelectCellWithForm() or TransformHKL() refs #5587

Changeset: 1ea62732d0f601b74f1e3dcaa404d737df7edb09

comment:6 Changed 8 years ago by Dennis Mikkelson

RE: 5587, Fix problem with compile under VC

refs #5587

Changeset: e29047fbe0e6ea860229716d8e829c4aae8fc1b9

comment:7 Changed 8 years ago by Vickie Lynch

  • Status changed from verify to verifying
  • Tester set to Vickie Lynch

comment:8 Changed 8 years ago by Vickie Lynch

  • Status changed from verifying to closed

Worked for peaks from TOPAZ_3131

comment:9 Changed 8 years ago by Owen Arnold

Looks like the wiki documentation for this algorithm hasn't been changed since May.

comment:10 Changed 8 years ago by Dennis Mikkelson

Made minor update to docs on Wiki

comment:11 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 6433

Note: See TracTickets for help on using tickets.