Ticket #4224 (closed: fixed)

Opened 9 years ago

Last modified 5 years ago

Fix bug with error bars when plotting MatrixWorkspace data

Reported by: Martyn Gigg Owned by: Owen Arnold
Priority: blocker Milestone: Release 2.0
Component: Mantid Keywords:
Cc: Blocked By:
Blocking: Tester: Karl Palmen

Description

From Pascal

Dear Mantid team,

Our users and ourselves are having a good laugh with this new bug. It looks like the centre chosen for the middle of the error bars is 1/2 bin out.

Thanks,

P To reproduce:

  • Go to View->Preferences->2D Plots->Curves Tab
  • Change the default curve style to "Line + Symbol" and change the symbol size to 1 and click Ok.
  • Load the attached file
  • Right-click on "Plot Spectrum With Error Bars"
  • Zoom in very close to the curve and you will see the error bars half-way between the points whereas they should be on the points

Attachments

test-curve.nxs (1.9 MB) - added by Martyn Gigg 9 years ago.
Screenshot-6.png (211.2 KB) - added by Martyn Gigg 9 years ago.

Change History

Changed 9 years ago by Martyn Gigg

Changed 9 years ago by Martyn Gigg

comment:1 Changed 9 years ago by Martyn Gigg

Refs #4224. Fix a bug with plotting error bars.

Commit b208d0fecca770d54d64f10a4fbdc91ea864a084 introduced the bug as it removed the itemChanged overload that is necessary for MatrixWorkspace curves.

Changeset: 3800489c4b7406b3e99019e7cd1d3ef58abd919a

comment:2 Changed 9 years ago by Nick Draper

  • Status changed from new to assigned
  • Owner set to Owen Arnold
  • Priority changed from major to blocker

comment:3 Changed 9 years ago by Owen Arnold

  • Status changed from assigned to accepted

comment:4 Changed 9 years ago by Owen Arnold

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

Martyn implemented the fix.

I've tested out the given schenario and the error bars are now back to being dead-center. Plotting of collapsed IMDWorkspaces is also working as expected.

comment:5 Changed 9 years ago by Karl Palmen

  • Status changed from verify to verifying
  • Tester set to Karl Palmen

comment:6 Changed 9 years ago by Karl Palmen

  • Status changed from verifying to closed

comment:7 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 5071

Note: See TracTickets for help on using tickets.