Ticket #2687 (assigned)
SANS Diagnostics Display Time - implement region
| Reported by: | Nick Draper | Owned by: | Peter Parker |
|---|---|---|---|
| Priority: | minor | Milestone: | Backlog |
| Component: | SANS | Keywords: | |
| Cc: | Blocked By: | #9131 | |
| Blocking: | Tester: |
Description
It would be nice to be able to also specify a region for Display Time, as H64-H68 or V0-V3 or H60-H68+V64-V66 for a “box†in the same way we use them in our user files; except we use H64>H68+V64>V66 there, so would be nice here, though not standard mantid convention, to use > or – here for “from†and “to†and keep + for combining regions.
Change History
comment:2 Changed 9 years ago by Nick Draper
- Milestone changed from Iteration 29 to Iteration 30
"New" tickets moved at the code freeze of iteration 29
comment:3 Changed 9 years ago by Steve Williams
- Owner changed from Steve Williams to Anders Markvardsen
- Status changed from new to assigned
comment:4 Changed 9 years ago by Nick Draper
- Milestone changed from Iteration 30 to Iteration 31
Bulk move of tickets to iteration 31 at the iteration 30 code freeze
comment:5 Changed 9 years ago by Nick Draper
- Milestone changed from Iteration 32 to Iteration 33
Moved to iteration 33 at iteration 32 code freeze
comment:6 Changed 8 years ago by Nick Draper
- Milestone changed from Release 2.1 to Release 2.2
Moved at end of release 2.1
comment:8 Changed 8 years ago by Anders Markvardsen
- Owner changed from Anders Markvardsen to Gesner Passos
comment:9 Changed 8 years ago by Nick Draper
- Milestone changed from Release 2.3 to Release 2.4
Moved to milestone 2.4
comment:10 Changed 8 years ago by Nick Draper
- Milestone changed from Release 2.4 to Release 2.5
Moved at the code freeze for release 2.4
comment:11 Changed 7 years ago by Nick Draper
- Milestone changed from Release 2.5 to Release 2.6
comment:14 Changed 7 years ago by Nick Draper
- Milestone changed from Release 2.6 to Backlog
Moved to backlog at the code freeze for R2.6
comment:17 Changed 7 years ago by Nick Draper
- Status changed from new to assigned
Bulk move of tickets out of triage (new) to assigned at the introduction of the triage state
comment:18 Changed 7 years ago by Gesner Passos
- Blocked By 9131 removed
As first step, it should implement #9131. Give the TimeIntegral a default operation.
The second step is to correct its behavior, or at least, document that it behaves differently from the previous 2 commnads. HorizontalIntegral and VerticalIntegral defines the ranges as Columns and Rows. While, TimeIntegral uses the input as spectrum numbers + the first spectrum of the bank. Hence, you can plot the TimeIntegral of 36864, but you can not plot VerticalIntegral of 193.
It is first necessary to ask Richar and Steven what they want. Do they want to sum the time channels for boxes (as suggested in the description of this ticket), for lines and for rows? Or they want to be able to access spectrum directly?
BTW, the time integral seems to be useless given that we have this functionality in InstrumentView, with much better user interactivity. This should be discussed as well.
comment:20 Changed 7 years ago by Peter Parker
Once #9131 is done, we will just remove the range field entirely as a fix here.
comment:21 Changed 7 years ago by Nick Draper
- Owner changed from Gesner Passos to Peter Parker
Ownership handed over to Peter
comment:23 Changed 5 years ago by Stuart Campbell
This ticket has been transferred to github issue 3534
Bulk move of tickets at the end of iteration 28