Ticket #777 (closed: fixed)

Opened 11 years ago

Last modified 5 years ago

Group detectors extension for the excitations group

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

Description

The spectra in the output workspace are sums of spectra in the input workspace. Which spectra are added to create which spectra depends a map file. As an example, a map file containing the information below will create an ouput workspace where the first spectrum is the sum of 1,2,4 and the second spectrum is the sum of 5,10,11,12,13,14,15.

1 1,2,4 2 5,10-15

Steve has asked for an example file from the excitiations group

Attachments

mid_tubes.map (2.4 KB) - added by Steve Williams 11 years ago.
map file that creates 14 spectra from ~700

Change History

comment:1 Changed 11 years ago by Steve Williams

  • Owner set to Nick Draper

Changed 11 years ago by Steve Williams

map file that creates 14 spectra from ~700

comment:2 Changed 11 years ago by Steve Williams

CORRECTION to the file format, I've attached an example map files and the format appears to be (with each name representing a single ASCII integer):

total_number_of_output_spectra output_spectra_number1 number_of_input_spectra1 input_spec1 input_spec2 input_spec3 input_spec4 input_spec5 input_spec6 output_spectra_number2 number_of_input_spectra2 input_spec1 input_spec2 input_spec3 input_spec4 input_spec5 input_spec6 input_spec7 input_spec8 output_spectra_number3 number_of_input_spectra3 input_spec1 input_spec2 ...

Specifing input spectra ranges with "-" is also supposed to be supported

comment:3 Changed 11 years ago by Steve Williams

That format again with line breaks

total_number_of_output_spectra
output_spectra_number1
number_of_input_spectra1
input_spec1 input_spec2 input_spec3 input_spec4
input_spec5 input_spec6
output_spectra_number2
number_of_input_spectra2
input_spec1 input_spec2 input_spec3 input_spec4
input_spec5 input_spec6 input_spec7 input_spec8
output_spectra_number3
number_of_input_spectra3
input_spec1 input_spec2
...

comment:4 Changed 11 years ago by Steve Williams

  • Owner changed from Nick Draper to Steve Williams

comment:5 Changed 11 years ago by Steve Williams

  • Status changed from new to assigned

This could take 2 days to implement including tests because it is complicated in these ways

  • the user specification has two variants, using comas or dashes
  • the user data will require some checking
  • some syntax errors should be fed back to the users so they can see to correct the problem

It should be finished on Wednesday 26 August

comment:6 Changed 11 years ago by Steve Williams

(In [3219]) GroupDetectors now accepts groups specified in a file as a list of spectra numbers. Still to do: let the algorithm interpret 1-3 as 1,2,3 and update the WIKI refs #777

comment:7 Changed 11 years ago by Steve Williams

(In [3227]) GroupDetectors attempted to fix tests by using the full path when loading the test input file refs #777

comment:8 Changed 11 years ago by Nick Draper

  • Milestone changed from Unassigned to Iteration 20

comment:9 Changed 11 years ago by Steve Williams

(In [3231]) Fixed tests that were failing because it couldn't open an input file, now it creates the file before loading it refs #777

comment:10 Changed 11 years ago by Steve Williams

(In [3244]) GroupDetectors now accepts ranges of integers specified like 11-120 and there are unit tests. Testing is ongoing and the WIKI needs updating refs #777

comment:11 Changed 11 years ago by Steve Williams

(In [3306]) LoadDetectorInfo is a new algorithm that adds information like gas pressure and offset times to the detector propertymap. Minor fixes for GroupDetectors and a the start of a script that will correct for detector efficiency variation refs #941 and #777 and #841

comment:12 Changed 11 years ago by Steve Williams

  • Status changed from assigned to testing
  • Resolution set to fixed

comment:13 Changed 11 years ago by Russell Taylor

  • Status changed from testing to closed

Tested OK by Martyn.

comment:14 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 1625

Note: See TracTickets for help on using tickets.