Ticket #4591 (closed: fixed)

Opened 9 years ago

Last modified 5 years ago

Disable Rebinning Button

Reported by: Owen Arnold Owned by: Michael Reuter
Priority: major Milestone: Release 2.0
Component: VATES Keywords:
Cc: russell.ewings@… Blocked By:
Blocking: Tester: Owen Arnold

Description

Rebinning should only be available for full MDWorkspaces in the SimpleVatesGUI. Currently the rebinning button is available in the standard view for MDHistoWorkspaces and it causes an exception to be thrown if clicked.

Attachments

error.png​ (132.8 KB) - added by Owen Arnold 9 years ago.

Change History

comment:1 Changed 9 years ago by Owen Arnold

  • Component changed from Mantid to VATES

comment:2 Changed 9 years ago by Owen Arnold

  • Summary changed from Diable Rebinning Button to Disable Rebinning Button

comment:3 Changed 9 years ago by Michael Reuter

  • Status changed from new to accepted
  • Milestone changed from Iteration 33 to Iteration 32

comment:4 Changed 9 years ago by Michael Reuter

Refs #4591. Disable rebin button for MDHistoWorkspaces.

Changeset: 7a42e35d8e5c69bbd1b93e4c0b330176d8a6db9c

comment:5 Changed 9 years ago by Michael Reuter

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

To test, load any MDHistoWorkspace into the simple interface from MantidPlot and check to see that the Rebin button on the view is disabled.

comment:6 Changed 9 years ago by Owen Arnold

  • Status changed from verify to verifying
  • Tester set to Owen Arnold

Changed 9 years ago by Owen Arnold

comment:7 Changed 9 years ago by Owen Arnold

See error.png. This ticket has not been fixed yet.

comment:8 Changed 9 years ago by Michael Reuter

  • Status changed from verifying to reopened
  • Resolution fixed deleted

This is a Windows only issue. Stupid Windows. It works fine on Linux.

comment:9 Changed 9 years ago by Michael Reuter

  • Status changed from reopened to accepted

comment:10 Changed 9 years ago by Michael Reuter

The issue seems to be that using typeid on Windows to find out the type of IMDWorkspace only returns garbage characters to the string. On Linux, this works just fine. Owen is trying to figure out what to do differently.

comment:11 Changed 9 years ago by Michael Reuter

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

Thanks to #4655, this should now work on Windows too.

comment:12 Changed 9 years ago by Owen Arnold

  • Status changed from verify to verifying

comment:13 Changed 9 years ago by Owen Arnold

  • Cc russell.ewings@… added
  • Status changed from verifying to closed

This works now and will be in the current release. Russell Ewings also highlighted this issue in his test feedback document.

comment:14 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​5438

Note: See TracTickets for help on using tickets.