Ticket #3381 (closed: worksforme)

Opened 9 years ago

Last modified 5 years ago

User script not converting X values properly

Reported by: Martyn Gigg Owned by: Andrei Savici
Priority: critical Milestone: Release 2.0
Component: Mantid Keywords:
Cc: Blocked By:
Blocking: Tester: Michael Reuter

Description

Spencer Howells has a script that performs some processing using CreateWorkspace and the final output has the first 2 spectra with different X values when they should be the same as the others.

Scripts are attached and the result workspace is called 'osi92711_Correct_osi92756'. The resulting plot should have the peaks at the same X position however 1 is offset as can be seen in the image.

Attachments

IncorrectConversion.png (15.5 KB) - added by Martyn Gigg 9 years ago.
osi92711_graphite004_red.nxs (1.3 MB) - added by Martyn Gigg 9 years ago.
osi92756_graphite004_red.nxs (1.1 MB) - added by Martyn Gigg 9 years ago.
CorT.py (334 bytes) - added by Martyn Gigg 9 years ago.
CorrTest.py (4.3 KB) - added by Martyn Gigg 9 years ago.
working.png (2.2 MB) - added by Andrei Savici 9 years ago.
My result seems OK

Change History

Changed 9 years ago by Martyn Gigg

Changed 9 years ago by Martyn Gigg

Changed 9 years ago by Martyn Gigg

Changed 9 years ago by Martyn Gigg

Changed 9 years ago by Martyn Gigg

comment:1 Changed 9 years ago by Martyn Gigg

In [13265]:

Add the exports for ISpectrum so that when a user has created a workspace with CreateWorkspace they can set up the necessary spectrum/detector links if the default is not right. Refs #3381

comment:2 Changed 9 years ago by Nick Draper

  • Priority changed from major to critical
  • Status changed from new to assigned
  • Owner set to Anyone

comment:3 Changed 9 years ago by Nick Draper

  • Milestone changed from Iteration 30 to Iteration 31

Bulk move of tickets to iteration 31 at the iteration 30 code freeze

comment:4 Changed 9 years ago by Andrei Savici

  • Owner changed from Anyone to Andrei Savici
  • Status changed from assigned to accepted

Changed 9 years ago by Andrei Savici

My result seems OK

comment:5 Changed 9 years ago by Andrei Savici

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

comment:6 Changed 9 years ago by Andrei Savici

just change

import CorrTestFirst as Main

to

import CorrTest as Main

comment:7 Changed 9 years ago by Michael Reuter

  • Status changed from verify to verifying
  • Tester set to Michael Reuter

comment:8 Changed 9 years ago by Michael Reuter

  • Status changed from verifying to closed

With the above suggestion, the script seems to generate the correct output. Note that this script does not work with the new Python API.

comment:9 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 4228

Note: See TracTickets for help on using tickets.