Ticket #10528 (closed: fixed)

Opened 6 years ago

Last modified 5 years ago

When plotting a WorkspaceGroup use the group name as the title

Reported by: Dan Nixon Owned by: Federico M Pouzols
Priority: major Milestone: Release 3.3
Component: GUI Keywords:
Cc: Blocked By:
Blocking: Tester: Dan Nixon

Description

Currently one of the workspace names is used as the plot title (the first alphabetically I believe), it would make more sense for this to be the name of the workspace group.

Change History

comment:1 Changed 6 years ago by Federico M Pouzols

  • Status changed from new to assigned
  • Owner set to Federico M Pouzols
  • Milestone changed from Backlog to Release 3.3

comment:2 Changed 6 years ago by Federico Montesino Pouzols

  • Status changed from assigned to inprogress

when plotting workspace groups, use group name as title, re #10528

Changeset: a4254bfa735741975cdc5856c9f580908844cf2f

comment:3 Changed 6 years ago by Federico Montesino Pouzols

ensure pointer is null, not needed but doesn't hurt, re #10528

Changeset: 33088257e5ec92e70dadb048de63edd077230dac

comment:4 Changed 6 years ago by Federico Montesino Pouzols

leave def constructor do its job, for broken compilers, re #10528

Changeset: 2ce19733e297c191e805b8ed00a08637f93c19d3

comment:5 Changed 6 years ago by Federico Montesino Pouzols

proper boost shared ptr assignment, re #10528

Changeset: 296ef344a90a7de390643209e7732a6f87899625

comment:6 Changed 6 years ago by Federico Montesino Pouzols

fix unreferenced variable warning, re #10528

Changeset: e40daa63dbf80c87a840bb83d0dd12206f721dc5

comment:7 Changed 6 years ago by Federico M Pouzols

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

This changes the behavior of plot spectrum (with/without error bars). For color fill plot the behavior doesn't change, it will make a plot for every workspace in the group and use the individual names as titles.

To test:

Take one or more workspace groups:

  • from files (for example TrainingCourseData/MUSR00015189.nxs or Test/AutoTestData/CSP78173.raw)
  • or create your own by grouping workspaces.

Check that when you select a group and plot spectra the title (plot title and window title) shows the group name and not the individual workspace names (don't get confused with the "- sequence_number" appendix that is added to the window titles). Often the names of workspace groups are very similar, differing just by a "_number" appendix. If you want to avoid confusion with the name appendices you might want to rename the groups and/or members of the groups. If you do so beware that there are issues with renaming workspaces in groups (example: #10514). Tests should pass, especially the MantidPlot... tests.

comment:8 Changed 6 years ago by Dan Nixon

  • Status changed from verify to verifying
  • Tester set to Dan Nixon

comment:9 Changed 6 years ago by Dan Nixon

  • Status changed from verifying to closed

Merge remote-tracking branch 'origin/feature/10528_use_group_name_when_plotting_group'

Full changeset: f19c348014223d06951406b97d88e47924772491

comment:10 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 11370

Note: See TracTickets for help on using tickets.