Ticket #11415 (assigned)

Opened 6 years ago

Last modified 5 years ago

ISIS SANS GUI enhancement to Diagnostics Tab

Reported by: Anders Markvardsen Owned by: Anton Piccardo-Selg
Priority: major Milestone: Backlog
Component: SANS Keywords:
Cc: King, stephen.king@… Blocked By:
Blocking: Tester:

Description

Implement as described from email from Steve:

When you currently perform a horizontal or vertical integral, irrespective of what run, or range of rows/columns, you inspected, Mantid creates workspaces called

{detector}-detector-bank_H1 {detector}-detector-bank_V1

which is irritating if you want to compare different runs/ranges as you have to rename the workspaces each time before proceeding.

Please could we adopt a workspace naming more akin to that used for the reduced data, say

{runnum}-{detector}-detector-Hmin-Hmax

and similarly for V?

I realise that the range specifications can be quite complicated (and trying to implement such in a workspace name would be quite convoluted), hence I propose the above as a compromise.

The good news is that the H & V integrals do seem to be working as expected.

Change History

comment:1 Changed 5 years ago by Peter Parker

  • Owner changed from Peter Parker to Anton Piccardo-Selg
  • Status changed from new to assigned

Suggested as being a short-term "easy win" for Anton as he starts working on SANS full time.

As Steve suggests, a simple renaming is all that's necessary here, but we need to be careful that diagnostics is not expecting the current workspace naming convention to be be in place elsewhere in the code.

Anton, in the meeting today we talked about how it's not even clear what the vertical integral should mean for SANS2D. Not something to worry about in this ticket -- consider current functionality for LOQ only.

A check for system test coverage at the same time would be nice.

comment:2 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 12254

Note: See TracTickets for help on using tickets.