Ticket #10656 (closed: fixed)

Opened 6 years ago

Last modified 5 years ago

Feature: Unified color map style for slice viewer and VSI

Reported by: Anton Piccardo-Selg Owned by: Anton Piccardo-Selg
Priority: major Milestone: Release 3.4
Component: GUI Keywords: VATES
Cc: Blocked By:
Blocking: Tester: Martyn Gigg

Description

Currently the VSI and the slice viewer use different default color maps. The default of both should be the same or similar at least. (The slice viewer color map is hardcoded, while VSI loads its definitions from XMl files).

Attachments

Test Script for Background Color and Color Map Settings.docx (23.7 KB) - added by Anton Piccardo-Selg 6 years ago.
SampleDataSet.nxs (164.6 KB) - added by Anton Piccardo-Selg 6 years ago.
MDHisto_Larmor.nxs (1.2 MB) - added by Anton Piccardo-Selg 6 years ago.
MDEvent_Osiris.nxs (2.3 MB) - added by Anton Piccardo-Selg 6 years ago.
MDHisto_Osiris.nxs (164.6 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

  • Milestone changed from Release 3.3 to Release 3.4

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

  • Status changed from assigned to inprogress

Refs #10656 Adding user-settable default colormap and background color

Changeset: 0af140a233dea99897041f12e0ef3b3d0e7b8913

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

Refs #10656 Store bgnd and cmap settings

Changeset: 1d76022b7cc5e4d67872510f5cbd8308f1dec1d7

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

Refs #10656 Add Preference Window tabs

Changeset: f54f44db84f92a3c8f35b9416e1ba6156fa11df9

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

Refs #10656 Add common settings for VSI and SV

Changeset: 1aea210742af3a30f1c0adb87e049901ee3c0066

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

Refs #10656 Removed test for BackgroundRgbProvider

Changeset: 592f4d9d512720767a753c3aff8b6b49fe92ef5c

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

Refs #10656 Migrate initial VSI view to QSettings

Changeset: e2b28cb4aa875440f92e956a7a7da1a204a6e6d3

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

Refs #10656 Fix closing window bug and clean up

Changeset: a85e69a92b17f5a510eab87dc841778e4a4c1291

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

Refs #10656 Solving merge conflicts

Changeset: 8160a71819f3dc10199b810b83646ae6d8eded4d

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

Refs #10656 Fix for color maps

Changeset: c7411c3435cb1353221a7c48a787db2c969fdd45

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

Refs #10656 Workaround for bad color maps

Changeset: d02a2320d60d1149db349b484bb68e23ee8761a2

comment:13 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 #332.

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

Refs #10656 Fix header issues for color maps

Changeset: 34ffe45582200ebfe55de588a5c3af5cba394dad

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

Refs #10656 Changed the file stream input

Changeset: 41b1e2abcc669691e24f945d92952ae7626c62db

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

Refs #10656 Remove warnings through reordering

Changeset: 13132311429cc87d55942798d21cd5a02bbaa95c

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

Refs #10656 Fix order in slice viewer

Changeset: ba2ebd97dfeafba70e6cb59f95c9ab291c8ffe7c

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

Refs #10656 Fix for reordering

Changeset: 3546c6bddad1755d3e3c19b6668a61c2b628615b

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

Refs #10656 Resolved merge conflicts

Changeset: 2f1a2c58e05770ee853e26b19d467ad74606952e

Changed 6 years ago by Anton Piccardo-Selg

Changed 6 years ago by Anton Piccardo-Selg

Changed 6 years ago by Anton Piccardo-Selg

Changed 6 years ago by Anton Piccardo-Selg

Changed 6 years ago by Anton Piccardo-Selg

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

For Testing: Please see the attached document for testing. Please find the sample data sets attached. Please aske me for the MDHisto_Merlin data set, as it is too be to attach. I will pass it on to you.

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

Refs #10656 Fix cpp warnings

Changeset: d7de1b6078409c1ab44a7beef85667cfacd3cf5d

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

Refs #10656 Reorder in slice viewer

Changeset: 16a6a40a7f28c0c7da47fa76b51745f767abc299

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

Jenkins, retest this please

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

Jenkins, retest this please

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

Jenkins, restest this please

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

Refs #10656 Fix osx issue with file streams

Changeset: 1d745a8704a5471730c4b0e5d523f50845d483c8

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

Jenkins, retest this please

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

Jenkins, retest this please

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

Refs #10656 Fix intel compiler warnings

Changeset: dac3eb016a14959233cb2979c780c7da111b6080

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

Jenkins, retest this please

comment:31 Changed 6 years ago by Martyn Gigg

It looks like this can no longer be merged cleanly to master. Could you merge master to your branch, please?

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

Refs #10656 Resolve merge conflict

Changeset: 55ab31e572ecc5f2e725d6a823bf3fbb694aabb4

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

Refs #10656 Get rid of merge issue

Changeset: c6a974eaa988a5a1865544f0fb425f25d8dc6006

comment:34 Changed 6 years ago by Martyn Gigg

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

comment:35 Changed 6 years ago by Martyn Gigg

Some nice test instructions here. They seem to have covered all bases.

The functionality seems to make sense as far as I can tell. In future we may want to consider "importing" ParaView's colour map selector widget as we simply have a list of names that don't mean much to users but that is a future improvement.

comment:36 Changed 6 years ago by Martyn Gigg

  • Status changed from verifying to closed

Merge pull request #332 from mantidproject/feature/10656_unified_default_color_maps

Unified color maps and settings for Vsi

Full changeset: c969c245974b41d33b326a492ee1811524fab70d

comment:37 Changed 5 years ago by Nick Draper

  • Component changed from Framework to User Interface

comment:38 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 11498

Note: See TracTickets for help on using tickets.