Ticket #9056 (closed: fixed)

Opened 7 years ago

Last modified 5 years ago

After MantidTable gets replaced, all the columns in the updated view become Y-columns

Reported by: Arturs Bekasovs Owned by: Arturs Bekasovs
Priority: major Milestone: Release 3.2
Component: GUI Keywords:
Cc: Blocked By:
Blocking: Tester: Gesner Passos

Description

As we are unable to set column plot types using Python API at the present moment, the only way to reproduce it I've came up with is using the MuonAnalysis interface (where it was noticed in the first place):

  1. In MuonAnalysis, load any data set.
  2. On Data Analysis tab, fit it to something.
  3. On Results Table tab, select a few log values and create a table. Notice that column plot types should be set properly, i.e. log values to X or Label, fit parameters to Y and errors to YErr.
  4. Now try to create the table again, replacing the previous one. Notice that now all the columns have the type Y.

Change History

comment:1 Changed 7 years ago by Arturs Bekasovs

  • Component changed from Muon to User Interface

I guess it's not a Muon specific thing, it's just that it's the only way I've came up with to reproduce it.

comment:2 Changed 7 years ago by Nick Draper

  • Status changed from new to assigned

comment:3 Changed 7 years ago by Arturs Bekasovs

  • Status changed from assigned to inprogress

Refs #9056. Return the behaviour changed in #8774.

If tables are not cleared before filling, we get problems.

Changeset: 437a561e0c582cd271f10821ab55c09644bf672f

comment:4 Changed 7 years ago by Arturs Bekasovs

Refs #9056. Move column type setting to fill function.

Otherwise column types are set on creating the table and not on when it gets updated.

Changeset: 6017db14bebcec8cec923d825b782a2086225be4

comment:5 Changed 7 years ago by Arturs Bekasovs

Refs #9056. Minor style improvements.

Changeset: da0bd4172ada643eff88b97e6d4ae1fdfa32f933

comment:6 Changed 7 years ago by Arturs Bekasovs

Tester:

Make sure the problem is fixed. Column types should be set properly when the table is both created and updated.

comment:7 Changed 7 years ago by Arturs Bekasovs

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

comment:8 Changed 7 years ago by Gesner Passos

  • Status changed from verify to verifying
  • Tester set to Gesner Passos

comment:9 Changed 7 years ago by Gesner Passos

  • Status changed from verifying to closed

Merge remote-tracking branch 'origin/feature/9056_mantid_table_replace_all_y_columns'

Full changeset: 6d71488a57f27d4b7c959b1c079cdc09da672e2c

comment:10 Changed 7 years ago by Gesner Passos

It does correctly configure the labels and allows removing entries as well.

comment:11 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 9899

Note: See TracTickets for help on using tickets.