Ticket #6237 (closed: fixed)

Opened 8 years ago

Last modified 5 years ago

SANS: Adding SANS2D event data in GUI

Reported by: Anders Markvardsen Owned by: Gesner Passos
Priority: major Milestone: Release 2.4
Component: MantidPlot Keywords:
Cc: sarah.rogers@… Blocked By:
Blocking: Tester: Anders Markvardsen

Description

Sarah writes:

I checked with the script and adding files together using this method seems to give the expected answer. Attached are a few pdfs on the data which I hope are clear enough. Basically the intensity of the gui added file seems to be all over the place where it is higher than the unadded data sets at high and low Q but is lower than expected in mid Q. I’ve also attached a copy of the ‘adding script’. The mask file for the data was MASKSANS2D_123T_4m_Xpress_8mm.txt

The script which gives the correct answer is attached as SANSadd_standalone.py

Attached also is mask file MASKSANS2D_123T_4m_Xpress_8mm.txt

A number of pdf also added. For example added_data_2_zoomb.pdf shows a zoomed comparison of adding SANS2D run 16201 and 16187

This ticket is get the GUI adding to work as well as the script adding

Attachments

SANSadd_standalone.py (2.5 KB) - added by Anders Markvardsen 8 years ago.
script
MASKSANS2D_123T_4m_Xpress_8mm.txt (2.0 KB) - added by Anders Markvardsen 8 years ago.
mask file Sarah used
added_data_1.pdf (41.4 KB) - added by Anders Markvardsen 8 years ago.
added_data_2.pdf (37.8 KB) - added by Anders Markvardsen 8 years ago.
added_data_2_zoom.pdf (32.5 KB) - added by Anders Markvardsen 8 years ago.
added_data_2_zoomb.pdf (31.4 KB) - added by Anders Markvardsen 8 years ago.
sansadd2.py.patch (1011 bytes) - added by Gesner Passos 8 years ago.
This patch makes the output to be the same

Change History

Changed 8 years ago by Anders Markvardsen

script

Changed 8 years ago by Anders Markvardsen

mask file Sarah used

Changed 8 years ago by Anders Markvardsen

Changed 8 years ago by Anders Markvardsen

Changed 8 years ago by Anders Markvardsen

Changed 8 years ago by Anders Markvardsen

Changed 8 years ago by Gesner Passos

This patch makes the output to be the same

comment:1 Changed 8 years ago by Gesner Passos

  • Status changed from new to accepted

comment:2 Changed 8 years ago by Gesner Passos

Change SANSadd2 to skip dealing with event file for a while

In order to give the answer the user expected, this turn off dealing the event mode file. It is necessary to look into the reason why summing the event based data does not provide a good result. re #6237

Changeset: 95435e68e0aa08cd5fc6073c5ce0fbe6cd29f0a5

comment:3 Changed 8 years ago by Gesner Passos

Change SANSadd2 to skip dealing with event file for a while

In order to give the answer the user expected, this turn off dealing the event mode file. It is necessary to look into the reason why summing the event based data does not provide a good result. re #6237

Changeset: 60397d6898f02e6f0a7f740e09bc04fbbfd2566b

comment:4 Changed 8 years ago by Nick Draper

  • Milestone changed from Release 2.4 to Release 2.5

Moved at the code freeze for release 2.4

comment:5 Changed 8 years ago by Gesner Passos

  • Milestone changed from Release 2.5 to Release 2.4

comment:6 Changed 8 years ago by Gesner Passos

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

to Tester:

From Mantid menu: Interfaces -> SANS ISIS Add the file MASKSANS2D_123T_4m_Xpress_8mm.txt​, to User File. Change Tab: Add Runs: 16201 and 16187

The result should be exactly the one the user ask. Hint: You may apply her algorithm and match workspaces.

comment:7 Changed 8 years ago by Anders Markvardsen

  • Status changed from verify to verifying
  • Tester set to Anders Markvardsen

comment:8 Changed 8 years ago by Anders Markvardsen

  • Status changed from verifying to closed

Followed test and also find match

comment:9 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 7083

Note: See TracTickets for help on using tickets.