Ticket #1427 (accepted task)

Opened 6 weeks ago

Last modified 6 days ago

Create 2 new "parameters.xml"s and remove all instrument specific data into classes

Reported by: Steve Williams Owned by: Steve Williams
Priority: major Milestone: Unassigned
Keywords: Cc: martyn.gigg@…
Tester: Martyn Gigg

Description

  1. Create SANS2D_Parameters.xml and LOQ_Parameters.xml files with instrument dependent numbers, including monitor numbers, which spectra numbers are in which detectors and how this varies between run numbers, filename zero padding, defaults (transmission wavelengths for normalisation, monitor number selection, sample thinkness, etc.)
  2. Move loading and holding these data from SANSReduction.py into an instrument class.


Williams time estimate 1 day

Change History

Changed 6 weeks ago by Steve Williams

(In [5387]) Removing bad comments from SANSReduction.py re #1427

Changed 5 weeks ago by Steve Williams

(In [5434]) Extends the SANS Instrument class and there are now parameter files for LOQ and SANS2D which have removed some instrument specific constants from SANSReduction.py, although more remain re #1427

Changed 5 weeks ago by Steve Williams

(In [5435]) System test fix for change for the parameter map mis-match re #1427

Changed 5 weeks ago by Steve Williams

(In [5445]) Fix broken system tests -two small errors in the SANS scripts re #1427

Changed 5 weeks ago by Steve Williams

(In [5463]) System test fix for SANS instrument parameter files causing a parameter map mis-match with existing test data re #1427

Changed 5 weeks ago by Steve Williams

(In [5492]) The SANS instrument parameter files and the SANS Python Instrument class contain everything they need re #1427

Changed 5 weeks ago by Steve Williams

(In [5499]) Fix SANS system test by updating the parameter map in the comparison file re #1427

Changed 5 weeks ago by Steve Williams

  • status changed from new to accepted

Changed 5 weeks ago by Martyn Gigg

M Gigg time estimate: 0.5 days

Changed 5 weeks ago by Steve Williams

  • status changed from accepted to verify

Took 2 days of effort to complete

Changed 2 weeks ago by Nick Draper

  • resolution set to fixed

fill in missing resolution

Changed 2 weeks ago by Martyn Gigg

  • status changed from verify to verifying
  • tester set to Martyn Gigg

Changed 2 weeks ago by Martyn Gigg

  • status changed from verifying to closed

Confirmed that files exists and contain appropriate data. Verified that the scripts now use these values and not those stored in the scripts themselves.

Verified in version #5882

Changed 12 days ago by Steve Williams

  • priority changed from critical to major
  • status changed from closed to reopened
  • resolution fixed deleted
  • milestone changed from Iteration 24 to Unassigned

Changed 12 days ago by Steve Williams

  • status changed from reopened to accepted

Changed 6 days ago by Steve Williams

(In [6073]) Fix broken system tests (due to a small change to a parameter file) re #1427

Changed 6 days ago by Steve Williams

(In [6076]) A new system test doesn't yet work, remove it re #1427

Changed 6 days ago by Steve Williams

(In [6085]) Fix broken system test re #1427

Note: See TracTickets for help on using tickets.