Ticket #5162 (closed: fixed)

Opened 8 years ago

Last modified 5 years ago

Indirect CTE: Ability to specify default grouping file in instrument parameter file.

Reported by: Stuart Campbell Owned by: Stuart Campbell
Priority: major Milestone: Release 2.1
Component: Mantid Keywords:
Cc: peter.parker@… Blocked By:
Blocking: Tester: Peter Parker

Description

Add the ability to specify the default grouping file in the instrument parameter file.

Change History

comment:1 Changed 8 years ago by Stuart Campbell

  • Status changed from new to accepted

comment:2 Changed 8 years ago by Stuart Campbell

Refs #5162. Added properties for the mask and grouping directories.

Added groupingFiles.directory property. Added maskFiles.directory property.

Changeset: a3d26ea352ca38f9b8b617f8dd0c2c0131d1a4b7

comment:3 Changed 8 years ago by Stuart Campbell

Refs #5162. Added grouping parameters to BASIS parameter file.

Added Workflow.GroupingMethod Added Workflow.GroupingFile

Changeset: 9dddd874947d3468049f85523f620eb0d0dcc76d

comment:4 Changed 8 years ago by Stuart Campbell

Refs #5162. Will use grouping file from parameter file.

It will now check for a 'File' grouping method and if set it will then read the Workflow.GroupingFile parameter and set this to be the grouping_policy

Changeset: b6dd74515538993fe920d535cae0e8c582518381

comment:5 Changed 8 years ago by Stuart Campbell

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

To Test:

  • Start the Convert To Energy GUI
  • Select BASIS
  • Enter a BASIS (event) nexus filename
  • Run

Now look at the history of the output workspace, it should contain a grouping step with the file BASIS_Grouping.xml and there should only be 56 histograms.

comment:6 Changed 8 years ago by Stuart Campbell

Refs #5162. Added properties for the mask and grouping directories.

Added groupingFiles.directory property. Added maskFiles.directory property.

Changeset: a3d26ea352ca38f9b8b617f8dd0c2c0131d1a4b7

comment:7 Changed 8 years ago by Stuart Campbell

Refs #5162. Added grouping parameters to BASIS parameter file.

Added Workflow.GroupingMethod Added Workflow.GroupingFile

Changeset: 9dddd874947d3468049f85523f620eb0d0dcc76d

comment:8 Changed 8 years ago by Stuart Campbell

Refs #5162. Will use grouping file from parameter file.

It will now check for a 'File' grouping method and if set it will then read the Workflow.GroupingFile parameter and set this to be the grouping_policy

Changeset: b6dd74515538993fe920d535cae0e8c582518381

comment:9 Changed 8 years ago by Peter Parker

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

comment:10 Changed 8 years ago by Peter Parker

  • Blocked By 5198 added

comment:11 Changed 8 years ago by Peter Parker

  • Blocked By 5198 removed

No longer blocked by 5198 - a clean install of Mantid solved the problem.

comment:12 Changed 8 years ago by Peter Parker

Stu, I've given this a go with BSS_11841_event.nxs from the AutoTestData folder, and while it does work, I get approx ~7000 errors reported to the log, of the following form:

Pixel w/ ID = 8192 Cannot Be Located
...
Pixel w/ ID = 15871 Cannot Be Located

Not sure if this is expected functionality or whether you want me to open another ticket for it. It does take my primitive PC a long while to display all those errors!

comment:13 Changed 8 years ago by Stuart Campbell

Yes, it is expected... it is because the BASIS NeXus files contain some detectors pixels that don't physically exist. If it is a problem for you then create a ticket for me on the next release.

comment:14 Changed 8 years ago by Peter Parker

  • Status changed from verifying to closed

I thought that might be the case. Now you've confirmed that it's expected, I dont have any particular problem with it.

comment:15 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 6008

Note: See TracTickets for help on using tickets.