Ticket #9235 (closed: fixed)

Opened 7 years ago

Last modified 5 years ago

LoadSassena: add default option to sort by momentum transfer

Reported by: Jose Borreguero Owned by: Jose Borreguero
Priority: major Milestone: Release 3.2
Component: Indirect Inelastic Keywords:
Cc: lynchve@… Blocked By:
Blocking: Tester: Vickie Lynch

Description (last modified by Jose Borreguero) (diff)

It is not uncommon to find that the order of the structure factors is not according to increasing Q-modulus, which will lead to analysis errors if unaware or forget about this fact. Thus, it is highly desirable to add a default option to sort the data as is loaded.

TESTER: download and uncompress attached file test.tar.gz under directory /tmp, then go to /tmp/test and execute script test.py in MantidPlot script window.

Group workspaces "x" and "y" should be created, the first with unsorted momentum transfer, the second one with sorted values. Compare workspaces x_qvectors and y_qvectors, as well as the order of the histograms in the other workspaces.

Attachments

test.tar.gz (582.6 KB) - added by Jose Borreguero 7 years ago.

Change History

comment:1 Changed 7 years ago by Jose Borreguero

  • Cc lynchve@… added
  • Description modified (diff)
  • Summary changed from LoadSassena: add default option to apply SortByQVectors to LoadSassena: add default option to sort by momentum transfer

comment:2 Changed 7 years ago by Jose Borreguero

Refs #9235 Inserted index sorting

Changeset: 3e6ebe4fc7a87eb092c6d4834faaa5cda0ecc830

Changed 7 years ago by Jose Borreguero

comment:3 Changed 7 years ago by Jose Borreguero

  • Description modified (diff)

comment:4 Changed 7 years ago by Jose Borreguero

  • Status changed from new to assigned

comment:5 Changed 7 years ago by Jose Borreguero

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

comment:6 Changed 7 years ago by Jose Borreguero

  • Tester set to lynchve@ornl.gov

TESTER: download and uncompress attached file test.tar.gz under directory /tmp, then go to /tmp/test and execute script test.py in MantidPlot script window.

Group workspaces "x" and "y" should be created, the first with unsorted momentum transfer, the second one with sorted values. Compare workspaces x_qvectors and y_qvectors, as well as the order of the histograms in the other workspaces.

comment:7 Changed 7 years ago by Vickie Lynch

  • Status changed from verify to verifying
  • Tester changed from lynchve@ornl.gov to Vickie Lynch

comment:8 Changed 7 years ago by Vickie Lynch

  • Status changed from verifying to closed

Merge remote branch 'origin/feature/9235_LoadSassena_SortByQVectors'

Full changeset: c9d909cc86f7e9921b0af6a01582cc942d4e9fdd

comment:9 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 10078

Note: See TracTickets for help on using tickets.