Ticket #11544 (closed: fixed)

Opened 5 years ago

Last modified 5 years ago

Improve handling of vertical axis in Matrix window

Reported by: Dan Nixon Owned by: Dan Nixon
Priority: major Milestone: Release 3.4
Component: GUI Keywords:
Cc: Blocked By:
Blocking: Tester: Martyn Gigg

Description

Currently if you view the data for a workspace that has a BinEdgeAxis as the vertical/spectrum axis the values you will see are the lower bin edge, this leads to confusion as you would expect this to be the bin centre given that this is what is shown for the X axis.

A good demonstration of this is irs26176_graphite002_sqw.nxs.

Change History

comment:1 Changed 5 years ago by Dan Nixon

  • Status changed from new to inprogress

Support BinEdgeAxis for the vertical axis

Refs #11544

Changeset: bf0adb5f1cfb44bf5e3ea7562efef5de85f6287d

comment:2 Changed 5 years ago by Dan Nixon

  • Owner set to Dan Nixon
  • Milestone changed from Backlog to Release 3.4

comment:3 Changed 5 years ago by Dan Nixon

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

This is being verified as pull request #575.

comment:4 Changed 5 years ago by Martyn Gigg

  • Status changed from verify to verifying
  • Tester set to Martyn Gigg

comment:5 Changed 5 years ago by Martyn Gigg

I agree, I think the lower bin edge is not at all sensible or consistent and it can only cause confusion.

comment:6 Changed 5 years ago by Martyn Gigg

The changes look sensible and I've tried a couple of different axis types and they all still look as expected.

comment:7 Changed 5 years ago by Dan Nixon

Support BinEdgeAxis for the vertical axis

Refs #11544

Changeset: bf0adb5f1cfb44bf5e3ea7562efef5de85f6287d

comment:8 Changed 5 years ago by Martyn Gigg

  • Status changed from verifying to closed

Merge pull request #575 from mantidproject/11544_handle_bin_edge_axis_on_v_axis

Show bin centre on vertical axis when axis is BinEdgeAxis

Full changeset: cc00c65d0d2174d8ed213dbdfb0b4d88aebfc9a4

comment:9 Changed 5 years ago by Dan Nixon

Support BinEdgeAxis for the vertical axis

Refs #11544

Changeset: f558cb800f4b391c980a4d54a629858be739cbe6

comment:10 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 12382

Note: See TracTickets for help on using tickets.