Ticket #3424 (closed: fixed)

Opened 9 years ago

Last modified 5 years ago

MantidPlot: "Select Data Range" option doesn't work

Reported by: Mathieu Doucet Owned by: Keith Brown
Priority: major Milestone: Release 3.0
Component: GUI Keywords:
Cc: Blocked By:
Blocking: Tester: Michael Reuter

Description

After plotting a spectrum, go in the Data menu then select "Select Data Range" (or do Alt-S). You will get a warning pop-up saying "All the curves on this plot are empty!"

Change History

comment:1 Changed 9 years ago by Nick Draper

  • Owner set to Anyone
  • Status changed from new to assigned

comment:2 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:3 Changed 9 years ago by Karl Palmen

  • Owner changed from Anyone to Karl Palmen

comment:4 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:5 Changed 8 years ago by Karl Palmen

I've found that the error arises in line 51 of RangeSelectorTool.cpp because the equality concerning RTTI is not satisfied.

comment:6 Changed 8 years ago by Karl Palmen

I see that the graph has RTTI type 1000 = User Item, while the Range Selector Tool expects a graph of RTTI type 4 = Curve. See http://qwt.sourceforge.net/class_qwt_plot_item.html .

comment:7 Changed 8 years ago by Karl Palmen

I tried out what would happen if the Range Selector Tool were to accept a User Item as a curve. This as expected fixed the "All the curves on this plot are empty!", but did not lead to a working range selector. It seems the the Range Select Tool does require a QwtPlotCurve and will not work with a MantidMatrixCurve as provided by a Matrix Workspace.

comment:8 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:9 Changed 8 years ago by Nick Draper

  • Milestone changed from Release 2.2 to Release 2.3

Moved at the end of release 2.2

comment:10 Changed 8 years ago by Nick Draper

  • Milestone changed from Release 2.3 to Release 2.4

Moved to milestone 2.4

comment:11 Changed 8 years ago by Karl Palmen

  • Owner changed from Karl Palmen to Anyone

comment:12 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:13 Changed 7 years ago by Nick Draper

  • Milestone changed from Release 2.5 to Release 2.6

Moved to r2.6 at the end of r2.5

comment:14 Changed 7 years ago by Nick Draper

  • Status changed from assigned to accepted
  • Owner changed from Anyone to Nick Draper

The toolbar option has been removed a few releases ago.

comment:15 Changed 7 years ago by Nick Draper

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

comment:16 Changed 7 years ago by Michael Reuter

  • Status changed from verify to verifying
  • Tester set to Michael Reuter

comment:17 Changed 7 years ago by Michael Reuter

  • Status changed from verifying to reopened
  • Resolution wontfix deleted

While the toolbar option may be gone, the menu option is not and still gives the above error.

comment:18 Changed 7 years ago by Nick Draper

  • Owner changed from Nick Draper to Anyone
  • Milestone changed from Release 2.6 to Backlog

comment:19 Changed 7 years ago by Nick Draper

  • Owner changed from Anyone to Keith Brown
  • Component changed from MantidPlot to User Interface

Remove the menu option and keyboard shortcut

comment:20 Changed 7 years ago by Keith Brown

  • Milestone changed from Backlog to Release 3.0

comment:21 Changed 7 years ago by Keith Brown

  • Status changed from reopened to inprogress

comment:22 Changed 7 years ago by Keith Brown

Removed the offending item from the UI

"Select Data Range" has now been removed from all toolbars, menus and shortcuts.

Refs #3424

Changeset: 0d1d1e6abe37c86fee0faa050e89871eb4ec6e91

comment:23 Changed 7 years ago by Keith Brown

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

comment:24 Changed 7 years ago by Michael Reuter

  • Status changed from verify to verifying

comment:25 Changed 7 years ago by Michael Reuter

  • Status changed from verifying to closed

This works and was merged, but the automatic closer didn't work.

comment:26 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 4271

Note: See TracTickets for help on using tickets.