Ticket #8395 (closed: fixed)

Opened 7 years ago

Last modified 5 years ago

Allow Interaction with Larmor data

Reported by: Gesner Passos Owned by: Gesner Passos
Priority: major Milestone: Release 3.1
Component: SANS Keywords:
Cc: anders.markvardsen@…, robert.dalgliesh@…, peter.parker@… Blocked By:
Blocking: #8366, #8414, #8438 Tester: Peter Parker

Description

There are a minimal of things that must be introduced to allow SANS ISIS interface to allow interaction with LARMOR data.

This ticket does not intend to support the larmor reduction as a whole, but to allow it to run. It will them allow us to start interaction with the user to check how the reduction is done and if it is acceptable.

Attachments

larmor_input.csv (391 bytes) - added by Gesner Passos 7 years ago.
MaskLarmor.txt (2.1 KB) - added by Gesner Passos 7 years ago.
create_larmor_data2.2.py (1.6 KB) - added by Gesner Passos 7 years ago.
create_larmor_data2.py (1.6 KB) - added by Gesner Passos 7 years ago.
Creation of Larmor data for reduction
DIRECTM1_15785_12m_31Oct12_v12.dat (8.0 KB) - added by Gesner Passos 7 years ago.

Change History

comment:1 Changed 7 years ago by Gesner Passos

  • Status changed from new to inprogress

re #8395: creation of Larmor Instrument

This commit introduces the Larmor instrument that is required to allow reducing Larmor data. It was not possible to check with the user the system coordinates that he wants to use and other things.

But, this commit is necessary to allow the user start interacting with the reduction to allow us to address all the particularities of the reduction for LARMOR.

Changeset: fe6617b8c16174c26cf1455d57ada204deb3292b

comment:2 Changed 7 years ago by Gesner Passos

  • Cc anders.markvardsen@…, robert.dalgliesh@…, peter.parker@… added

Changed 7 years ago by Gesner Passos

Changed 7 years ago by Gesner Passos

Changed 7 years ago by Gesner Passos

Changed 7 years ago by Gesner Passos

Creation of Larmor data for reduction

Changed 7 years ago by Gesner Passos

comment:3 Changed 7 years ago by Gesner Passos

re #8395: correct Larmor zero padding

By the files current available inside the archiver. It seems that the user plans to use 8 zero-padding.

Changeset: d9e2ed7a3662485fe273c1ce0bc208ee20336bc4

comment:4 Changed 7 years ago by Gesner Passos

re #8395: Correct SaveCanSAS support for LARMOR instrument

Changeset: 8af0b8373e155eeedf6a188cbcd19f996d7f999f

comment:5 Changed 7 years ago by Gesner Passos

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

Tester:

1 - First of all, it is necessary to create some data for LARMOR. I've used the create_larmor_data2.py to do so. It uses SANS2D data to populate LARMOR workspace with data, and them, save the data. It finally, creates the sample information directly inside the nexus file. This data is not really a simulation of LARMOR because they will use event mode data. But, it is a start point.

2 - Having the files created. It is possible to 'play' with the reduction. I've added the MaskFile used to reduce the original SANS2D data (it requires the direct file as well).

3 - Open ISIS interface

4 - Select LARMOR instrument

5 - Load the MaskFile

6 - Select batch mode

7 - Load the batch file

8 - Execute Reduce 1D.

9 - Check the results: larmor_complete_reduction, larmor_sample_can_reduction, larmor_sample_trans_reduction.

10 - Execute Reduce 2D.

11 - Open 73rear_2D_1.75_15.5 as Color Fill Plot

12 - You may try to change some settings inside 'Reduction Settings' Tab - the only one you can not expect sensible result is changing Detector bank combobox. Larmor has just one detector bank, so selecting front, both, merged will make the reduction to fail. It is something I will have to discuss what it is better doing about banks detector and LARMOR instrument. But, does not belong to this ticket.

13 - After changing settings, you may want to try the reduction again.

14 - Moving to Geometry Tab. You will be able to search for the centre. Try it, and check that it works nicely.

15 - You should be able to use the Add runs as well (for example to add runs 73 and 70).

No work for the others tabs was done here in this ticket.

Remember: This ticket is not to provide correct answer for the reduction, but to allow it to be run and not explode. It will them go into the interaction with the user to perform a correct reduction.

comment:6 Changed 7 years ago by Gesner Passos

  • Blocking 8414 added

comment:7 Changed 7 years ago by Gesner Passos

  • Blocking 8438 added

comment:8 Changed 7 years ago by Gesner Passos

re #8395: creation of Larmor Instrument

This commit introduces the Larmor instrument that is required to allow reducing Larmor data. It was not possible to check with the user the system coordinates that he wants to use and other things.

But, this commit is necessary to allow the user start interacting with the reduction to allow us to address all the particularities of the reduction for LARMOR.

Changeset: 0abe0e457cca1acdefcafb9c8f62d485be221302

comment:9 Changed 7 years ago by Gesner Passos

re #8395: correct Larmor zero padding

By the files current available inside the archiver. It seems that the user plans to use 8 zero-padding.

Changeset: c0806787f2ccedfd6d1af9a2a77508bab2dd7a19

comment:10 Changed 7 years ago by Gesner Passos

re #8395: Correct SaveCanSAS support for LARMOR instrument

Changeset: 075342c8782a9176f7d39cceca9dfe58106fbd61

comment:11 Changed 7 years ago by Nick Draper

  • Status changed from verify to verifying
  • Tester set to Nick Draper

comment:12 Changed 7 years ago by Nick Draper

  • Status changed from verifying to verify
  • Tester Nick Draper deleted

File creation script requires h5py, this is not installed by default, and would be much easier to install in a linux pc. Therefore returning to pool.

comment:13 Changed 7 years ago by Peter Parker

  • Status changed from verify to verifying
  • Tester set to Peter Parker

comment:14 Changed 7 years ago by Peter Parker

  • Status changed from verifying to closed

Merge remote-tracking branch 'origin/feature/8395_larmor_reduction_introduction'

Full changeset: 404e7be5972be08f24dc04bd4219d9520ef64f83

comment:15 Changed 7 years ago by Peter Parker

I thought I came across some errors when running the 1D and 2D reductions for the first time and then trying to search for the centre in the Geometry tab, but I can't seem to reproduce it.

Otherwise, the 15 steps seem to work fine for me.

comment:16 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 9240

Note: See TracTickets for help on using tickets.