Ticket #11330 (closed: fixed)

Opened 6 years ago

Last modified 5 years ago

Feature: Improve peak table view in splatterplot

Reported by: Anton Piccardo-Selg Owned by: Anton Piccardo-Selg
Priority: major Milestone: Release 3.4
Component: GUI Keywords: VSI
Cc: Blocked By:
Blocking: Tester: Owen Arnold

Description

Several things that Pascal suggested:

  1. Include sort function of table
  2. We are zooming in to much on the peaks (increase camera distance)
  3. Add ellipsoid axes
  4. Color for ellipsoid glyph of different tables (might not be possible)

Attachments

Test instructions for ticket 11330.docx (15.4 KB) - added by Anton Piccardo-Selg 6 years ago.

Change History

comment:1 Changed 6 years ago by Anton Piccardo-Selg

  • Status changed from new to assigned

comment:2 Changed 6 years ago by Anton Piccardo-Selg

Information from ParaView:

The coloring of an object (in this case, the result of the glyph filter) is controlled by the representation object. The representation object can be set to be either a solid color or mapping a variable. It sounds like your glyph representation is already set to a solid color since the default solid color is white.

There are several parameters on the representation object that control the solid color. The most relevant is the DiffuseColor parameter. Change that to be yellow (i.e. [1,1,0]) and that should do what you want.

comment:3 Changed 6 years ago by Anton Piccardo-Selg

Items 1, 2, and 4 are implemented. Item 3 could be tricky and should be addressed in a separete ticket.

Note regarding item 4: The parameter whcih needs to be set is not the DiffuseColor parameter but rather the AmbientColor parameter since we are dealing with a WireFrame representation of the peak source data.

comment:4 Changed 6 years ago by Anton Piccardo-Selg

  • Status changed from assigned to inprogress

Refs #11330 Sorting the peaks table in vsi

Changeset: e90f38af1d55270e76e853531e418d56290c8730

comment:5 Changed 6 years ago by Anton Piccardo-Selg

Refs #11330 Adding colored peak workspace in vsi

Changeset: bef88f4c33237d5f33c9ede9e704085b759f3ace

comment:6 Changed 6 years ago by Anton Piccardo-Selg

Refs #11330 Fixing doxygen warnings of Vates related stuff

Changeset: 9bc3be915be03d8099b9f560e2e44a40c5123e01

comment:7 Changed 6 years ago by Anton Piccardo-Selg

Refs #11330 Fix mismatch Qlab Qsample in vsi

Changeset: 2b9f2ccdba39f77790d10debbd8c3a07abc52327

comment:8 Changed 6 years ago by Anton Piccardo-Selg

Refs #11330 Fixes for improvement of peak table

Changeset: f3167c94a2e7dcea9a6cb6f4b16ad6b4ad708424

comment:9 Changed 6 years ago by Anton Piccardo-Selg

Refs #11330 Change diffuse color to ambient color

Changeset: 5aff933e95c45ff67992d0415f58134067dec6fa

comment:10 Changed 6 years ago by Anton Piccardo-Selg

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

This is being verified as pull request #547.

comment:11 Changed 6 years ago by Anton Piccardo-Selg

Refs #11330 Remove compiler warnings

Changeset: e0d1bdd5854715329b1f6fbbe7701894836d2879

Changed 6 years ago by Anton Piccardo-Selg

comment:12 Changed 6 years ago by Owen Arnold

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

comment:13 Changed 6 years ago by Anton Piccardo-Selg

Refs #11330 Fix missing labels in splatterplot

Changeset: 58731f103373b3a2d56f6ea4ae60466fab7d5495

comment:14 Changed 6 years ago by Anton Piccardo-Selg

Refs #11330 Make getters const for peaks presenter

Changeset: 59913199e48d6a38e17256c6d6151685699779a8

comment:15 Changed 5 years ago by Owen Arnold

  • Status changed from verifying to closed

Merge pull request #547 from mantidproject/feature/11330_improve_peak_table_view_splatterplot

Improve peak table view in the splatterplot in the vsi

Full changeset: 0511062ee33813e2e1b1beff0f5088be5203d04b

comment:16 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 12169

Note: See TracTickets for help on using tickets.