Ticket #1323 (closed: fixed)

Opened 10 years ago

Last modified 5 years ago

Allow the SANS analysis to use data from multi-peroid files

Reported by: Steve Williams Owned by: Anders Markvardsen
Priority: major Milestone: Iteration 20
Component: Mantid Keywords:
Cc: Blocked By:
Blocking: Tester:

Description

A copy of the requests for more multi-period support are below

Steve W,

Though that would work, it might be easier to have another column in the csv file for period number, this would then be very easy to set up in excel without tedious editing it a lot of periods were involved.

Steve K – what do you think ? (as you planned the csv file).

Richard

Dr.R.K.Heenan, SANS Instrument Scientist, ISIS Facility, R3 1-22, Rutherford Appleton Lab, HSIC, Didcot, OX11 0QX. (01235-446744, 07764-212155)

From: Williams, Steve (STFC,RAL,ISIS) Sent: 17 June 2010 11:41 To: Heenan, Richard (STFC,RAL,ISIS) Subject: RE: Multi-period data in Mantid

Richard,

Multi-period files aren't supported in batch mode. How would you like to specify period information? Maybe RUN_NUMBERpPERIOD (e.g. 5512p4) is a convenient way to do it, what is best for you?

Regards, Steve

Attachments

mantid_batch_2.csv (125 bytes) - added by Anders Markvardsen 9 years ago.
Demo batch csv file provided by Sarah
MASKSANS2D_112H_4m_Lu_stopflow.txt (1.4 KB) - added by Anders Markvardsen 9 years ago.
Demo user file provided by Sarah
DIRECT_RUN524_4m_06May11.dat (8.0 KB) - added by Anders Markvardsen 9 years ago.
Also needed for Sarah's demo

Change History

comment:1 Changed 10 years ago by Steve Williams

  • Status changed from new to accepted

comment:2 Changed 10 years ago by Steve Williams

  • Priority changed from critical to major
  • Milestone changed from Iteration 24 to Unassigned

comment:3 Changed 9 years ago by Anders Markvardsen

  • Owner changed from Steve Williams to Anders Markvardsen
  • Status changed from accepted to assigned
  • Component set to Mantid

Some functionality was added under ticket #3199. Discuss the requirements again with SANS?

Changed 9 years ago by Anders Markvardsen

Demo batch csv file provided by Sarah

Changed 9 years ago by Anders Markvardsen

Demo user file provided by Sarah

comment:4 Changed 9 years ago by Anders Markvardsen

  • Status changed from assigned to accepted

Sarah demonstrated multi-period ISIS SANS analysis using the attached files and where the data referred to in the attached .csv file is from the directory:


isis\inst$\ndxsans2d\instrument\cycle_11_2

She was happy except from one remaining bug. When the reduction runs, using the attached csv file as an example, the temperatory 9552p2_sans_nxs, 9552p3_sans_nxs etc are not deleted as the reduction takes place, which can cause a memory problem. Make sure these gets deleted as the reduction progress.

comment:5 Changed 9 years ago by Anders Markvardsen

Notice to test this ticket. Used the attached User file and .csv file. Add directory:

"
isis\inst$\NDXSANS2D\Instrument\data\cycle_11_2"

Tick in ISIS SANS 'Multi-period'. Select Batch mode dial which allow you to select csv file.

Changed 9 years ago by Anders Markvardsen

Also needed for Sarah's demo

comment:6 Changed 9 years ago by Anders Markvardsen

In [14004]:

Deleting additional temporary files when executing ISIS sans batch reduction. Re #1323

comment:7 Changed 9 years ago by Anders Markvardsen

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

Comment from instrument scientist:

I’ve just downloaded the latest Mantid and tested your fixes for the batch reducing of multiperiod data and it works perfectly!

comment:8 Changed 9 years ago by Nick Draper

  • Status changed from verify to closed

mass closed in cleanup

comment:9 Changed 7 years ago by Nick Draper

  • Milestone changed from Unassigned to Iteration 20

comment:10 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 2170

Note: See TracTickets for help on using tickets.