Ticket #8575 (closed: fixed)

Opened 7 years ago

Last modified 5 years ago

Update SANSBatchMode code to allow saving of WorkspaceGroups with the correct names

Reported by: Martyn Gigg Owned by: Martyn Gigg
Priority: major Milestone: Release 3.1
Component: SANS Keywords:
Cc: Blocked By:
Blocking: Tester: Russell Taylor

Description


Change History

comment:1 Changed 7 years ago by Martyn Gigg

  • Status changed from new to inprogress

Fix saving a WorkspaceGroup in SANSBatchMode

It previously just saved the last thing in the group with the group name. Now the group is unrolled and each workspace save individually. Refs #8575

Changeset: 1c99e7bb4e75c89a019681ae94d123eba56f80a8

comment:2 Changed 7 years ago by Martyn Gigg

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

Branch: feature/8575_save_workspace_groups_sans_batch

Tester: The following script requires the systemtests/Data/SANS2D directory to be in your data search path. After it has completed you should find a group called 5512_SANS2DBatch. Each member should have a name starting "5512". In you default save directory you should find a file that corresponds to each workspace in that group with the same name as the group member's name.

import ISISCommandInterface as ici
import SANSBatchMode as batch

#put your mask file in here
MASKFILE = FileFinder.getFullPath('MaskSANS2DReductionGUI.txt')
#put your batch file in here
BATCHFILE = FileFinder.getFullPath('SANS2D_multiPeriodTests.csv')

ici.SANS2D()
ici.MaskFile(MASKFILE)
 
#automatically 'ticks' RKH format but other options can be chosen from the gui
fit_settings = batch.BatchReduce(BATCHFILE, '.nxs',saveAlgs={'SaveRKH':'txt'}, combineDet='rear')

comment:3 Changed 7 years ago by Karl Palmen

  • Status changed from verify to verifying
  • Tester set to Karl Palmen

comment:4 Changed 7 years ago by Karl Palmen

  • Status changed from verifying to verify
  • Tester Karl Palmen deleted

When I ran the script, it was unable to understand 'ici'. I return to testing pool in case anyone has their system configured to understand 'ici'.

comment:5 Changed 7 years ago by Russell Taylor

  • Status changed from verify to verifying
  • Tester set to Russell Taylor

comment:6 Changed 7 years ago by Russell Taylor

  • Status changed from verifying to closed

Merge remote branch 'origin/feature/8575_save_workspace_groups_sans_batch'

Full changeset: cc2e6cf8708b5f2282f40ad2e938f11a95d5c5ca

comment:7 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 9418

Note: See TracTickets for help on using tickets.