Ticket #6901 (closed: fixed)

Opened 7 years ago

Last modified 5 years ago

Improve Usability of SCD GUI (MantidEV)

Reported by: Dennis Mikkelson Owned by: Dennis Mikkelson
Priority: major Milestone: Release 2.5
Component: Mantid Keywords:
Cc: petersonpf@…, mikkelsonr@… Blocked By:
Blocking: Tester: Vickie Lynch

Description

Add features to improve usability of MantidEV, specifically:

  1. Subdivide the MD workspace more so the slice viewer works better intially.
  2. Add unit labels to integration parameters.
  3. Add option to reset default values for the parameters.

plus additional basic enhancements that can be completed in time for the release.

Change History

comment:1 Changed 7 years ago by Dennis Mikkelson

RE 6901 MantidEV usability

Changed |Q| range to go up to 35. Set MinRecursionDepth to 7 on ConvertToMD, to give at least 128x128x128 boxes. This makes the slice viewer a bit more usable.

Refs #6901

Changeset: b95c8d52d326a19f0a42d971164493ca1d88261c

comment:2 Changed 7 years ago by Dennis Mikkelson

RE 6901 MantidEV usability

Added units (A-1) to appropriate labels on the Integrate tab.

refs #6901

Changeset: 9d6131e1be668d96d3c1ca3d0ce2b9876da2e3f3

comment:3 Changed 7 years ago by Dennis Mikkelson

RE 6901 MantidEV usability improvements

Added menu option to reset default state of all GUI components. Removed some debug prints.

refs #6901

Changeset: 4fcf217956a241147b6f8813842c29a6e0c92ca9

comment:4 Changed 7 years ago by Dennis Mikkelson

RE 6901 MantidEV usability

Changed |Q| range to go up to 35. Set MinRecursionDepth to 7 on ConvertToMD, to give at least 128x128x128 boxes. This makes the slice viewer a bit more usable.

Refs #6901

Changeset: b95c8d52d326a19f0a42d971164493ca1d88261c

comment:5 Changed 7 years ago by Dennis Mikkelson

RE 6901 MantidEV usability

Added units (A-1) to appropriate labels on the Integrate tab.

refs #6901

Changeset: 9d6131e1be668d96d3c1ca3d0ce2b9876da2e3f3

comment:6 Changed 7 years ago by Dennis Mikkelson

RE 6901 MantidEV usability improvements

Added menu option to reset default state of all GUI components. Removed some debug prints.

refs #6901

Changeset: 4fcf217956a241147b6f8813842c29a6e0c92ca9

comment:7 Changed 7 years ago by Dennis Mikkelson

RE 6901 MantidEV usability

Added controls on the first form to allow the user to restrict the range of Q values, and choose whether or not to do the Lorentz correction when converting to MD. This will be necessary for large molecule/protein crystallography.

refs #6901

Changeset: 0649a3e85ab59543ed9c3f33adf5abd1bdae127f

comment:8 Changed 7 years ago by Dennis Mikkelson

RE 6901 MantidEV usability

Added controls on the first form to allow the user to restrict the range of Q values, and choose whether or not to do the Lorentz correction when converting to MD. This will be necessary for large molecule/protein crystallography.

refs #6901

Changeset: 0649a3e85ab59543ed9c3f33adf5abd1bdae127f

comment:9 Changed 7 years ago by Dennis Mikkelson

RE 6901 MantidEV usability

Added help menu item and linked it to initial, incomplete help page for MantidEV. The help page will be completed before the release.

refs #6901

Changeset: d410c9c49ed36c13d4c2c87f9748ebebfae715ad

comment:10 Changed 7 years ago by Dennis Mikkelson

RE 6901 MantidEV usability

When the user selects an event file to load, default names are now provided for the event, md and peaks workspaces.

refs #6901

Changeset: 9999ef74cc68a01e734ac8c71c8cca76a8db2a3c

comment:11 Changed 7 years ago by Dennis Mikkelson

RE 6901 MantidEV usability

Added help menu item and linked it to initial, incomplete help page for MantidEV. The help page will be completed before the release.

refs #6901

Changeset: d410c9c49ed36c13d4c2c87f9748ebebfae715ad

comment:12 Changed 7 years ago by Dennis Mikkelson

RE 6901 MantidEV usability

When the user selects an event file to load, default names are now provided for the event, md and peaks workspaces.

refs #6901

Changeset: 9999ef74cc68a01e734ac8c71c8cca76a8db2a3c

comment:13 Changed 7 years ago by Dennis Mikkelson

RE 6901 MantidEV usability

Added option to append peaks to an existing peaks file, as requested by Xiaoping.

refs #6901

Changeset: f87f804a8fd81377dcc0754ab0f08cfc7a91ad60

comment:14 Changed 7 years ago by Dennis Mikkelson

RE 6901 MantidEV usability

Added option to append peaks to an existing peaks file, as requested by Xiaoping.

refs #6901

Changeset: f87f804a8fd81377dcc0754ab0f08cfc7a91ad60

comment:15 Changed 7 years ago by Dennis Mikkelson

  • Status changed from new to accepted

comment:16 Changed 7 years ago by Dennis Mikkelson

RE 6901 MantidEV usability

Minor change to .ui file, to fix layout on some systems after adding (A-1) to appropriate labels on the Integrate tab.

refs #6901

Changeset: bf36730f214094da8544636eb6bd01859a0e5487

comment:17 Changed 7 years ago by Dennis Mikkelson

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

NOTES TO TESTER:

The branch name for this ticket is: feature/6901_MantidEV_usability

The improvements made under this ticket can be tested as follows, after launching the SCD Event Data Reduction interface.

  1. Select the Help page. A browser should be directed to the help page on the WiKi. The help page is not complete, but will be completed in the next few days, before the release. Verify that the code to link to the page is done and works.
  1. Enter some random values in different controls, then choose the menu option File->Reset_Default_Settings. The controls should clear and reasonable values should be entered for the various numeric values.
  1. On the first tab, browse to select an SCD event data file, such as TOPAZ_3132.nxs in the system test data directory. After selecting the file to load, reasonable default names for the event, md and peaks workspaces will be set.
  1. Press apply to load the data and convert to MD. When that has finished, go to the Find Peaks tab and press Apply to find peaks. From the File menu, choose Save_Isaw_Peaks, and save the peaks to some file. If you choose Save_Isaw_Peaks a second time, you will get an option to Replace the file or Append the new peaks to the file. By doing this repeatedly and noting the file size on the disk, or examining the contents of the file, you can verify that the peaks are appended or replace the original file according to your choice. (In real use, this would be done with different runs to build up a complete set of peaks from all runs.)
  1. Note that the units (A-1) have been added to appropriate labels on the Integrate tab.
  1. If you bring up SliceViewer for the md workspace, note that there are no huge blocks, since the minimum number of subdivisions has been raised to 27.
  1. Going back to the first Select Data tab, reduce the Max |Q| to Map to MD to 10, and press Apply again. When this completes, Find Peaks again, and note that in the resulting peaks workspace, no component of the Sample Q exceeds the specified limit 10.
  1. To verify that the Lorentz correction option is working, set |Q| to 25, turn Lorentz correction ON, find 300 peaks, then find the UB using the FFT. 298/300 peaks should index. Go through this process again but with the Lorentz correction turned off. In this case only 246/300 peaks should index, verifying that the Lorentz correction helps find more valid peaks.

comment:18 Changed 7 years ago by Dennis Mikkelson

RE 6901 MantidEV usability

Changed |Q| range to go up to 35. Set MinRecursionDepth to 7 on ConvertToMD, to give at least 128x128x128 boxes. This makes the slice viewer a bit more usable.

Refs #6901

Changeset: b95c8d52d326a19f0a42d971164493ca1d88261c

comment:19 Changed 7 years ago by Dennis Mikkelson

RE 6901 MantidEV usability

Added units (A-1) to appropriate labels on the Integrate tab.

refs #6901

Changeset: 9d6131e1be668d96d3c1ca3d0ce2b9876da2e3f3

comment:20 Changed 7 years ago by Dennis Mikkelson

RE 6901 MantidEV usability improvements

Added menu option to reset default state of all GUI components. Removed some debug prints.

refs #6901

Changeset: 4fcf217956a241147b6f8813842c29a6e0c92ca9

comment:21 Changed 7 years ago by Dennis Mikkelson

RE 6901 MantidEV usability

Added controls on the first form to allow the user to restrict the range of Q values, and choose whether or not to do the Lorentz correction when converting to MD. This will be necessary for large molecule/protein crystallography.

refs #6901

Changeset: 0649a3e85ab59543ed9c3f33adf5abd1bdae127f

comment:22 Changed 7 years ago by Vickie Lynch

  • Status changed from verify to verifying
  • Tester set to Vickie Lynch

comment:23 Changed 7 years ago by Dennis Mikkelson

RE 6901 MantidEV usability

Changed |Q| range to go up to 35. Set MinRecursionDepth to 7 on ConvertToMD, to give at least 128x128x128 boxes. This makes the slice viewer a bit more usable.

Refs #6901

Changeset: b95c8d52d326a19f0a42d971164493ca1d88261c

comment:24 Changed 7 years ago by Dennis Mikkelson

RE 6901 MantidEV usability

Added units (A-1) to appropriate labels on the Integrate tab.

refs #6901

Changeset: 9d6131e1be668d96d3c1ca3d0ce2b9876da2e3f3

comment:25 Changed 7 years ago by Dennis Mikkelson

RE 6901 MantidEV usability improvements

Added menu option to reset default state of all GUI components. Removed some debug prints.

refs #6901

Changeset: 4fcf217956a241147b6f8813842c29a6e0c92ca9

comment:26 Changed 7 years ago by Dennis Mikkelson

RE 6901 MantidEV usability

Added controls on the first form to allow the user to restrict the range of Q values, and choose whether or not to do the Lorentz correction when converting to MD. This will be necessary for large molecule/protein crystallography.

refs #6901

Changeset: 0649a3e85ab59543ed9c3f33adf5abd1bdae127f

comment:27 Changed 7 years ago by Dennis Mikkelson

RE 6901 MantidEV usability

Minor change to .ui file, to fix layout on some systems after adding (A-1) to appropriate labels on the Integrate tab.

refs #6901

Changeset: bf36730f214094da8544636eb6bd01859a0e5487

comment:28 Changed 7 years ago by Dennis Mikkelson

RE 6901 MantidEV usability

Changed |Q| range to go up to 35. Set MinRecursionDepth to 7 on ConvertToMD, to give at least 128x128x128 boxes. This makes the slice viewer a bit more usable.

Refs #6901

Changeset: b95c8d52d326a19f0a42d971164493ca1d88261c

comment:29 Changed 7 years ago by Dennis Mikkelson

RE 6901 MantidEV usability

Added units (A-1) to appropriate labels on the Integrate tab.

refs #6901

Changeset: 9d6131e1be668d96d3c1ca3d0ce2b9876da2e3f3

comment:30 Changed 7 years ago by Dennis Mikkelson

RE 6901 MantidEV usability improvements

Added menu option to reset default state of all GUI components. Removed some debug prints.

refs #6901

Changeset: 4fcf217956a241147b6f8813842c29a6e0c92ca9

comment:31 Changed 7 years ago by Dennis Mikkelson

RE 6901 MantidEV usability

Added controls on the first form to allow the user to restrict the range of Q values, and choose whether or not to do the Lorentz correction when converting to MD. This will be necessary for large molecule/protein crystallography.

refs #6901

Changeset: 0649a3e85ab59543ed9c3f33adf5abd1bdae127f

comment:32 Changed 7 years ago by Vickie Lynch

  • Status changed from verifying to closed

Reduction still works and new features are there.

comment:33 Changed 7 years ago by Dennis Mikkelson

RE 6901 MantidEV usability

Changed |Q| range to go up to 35. Set MinRecursionDepth to 7 on ConvertToMD, to give at least 128x128x128 boxes. This makes the slice viewer a bit more usable.

Refs #6901

Changeset: b95c8d52d326a19f0a42d971164493ca1d88261c

comment:34 Changed 7 years ago by Dennis Mikkelson

RE 6901 MantidEV usability

Added units (A-1) to appropriate labels on the Integrate tab.

refs #6901

Changeset: 9d6131e1be668d96d3c1ca3d0ce2b9876da2e3f3

comment:35 Changed 7 years ago by Dennis Mikkelson

RE 6901 MantidEV usability improvements

Added menu option to reset default state of all GUI components. Removed some debug prints.

refs #6901

Changeset: 4fcf217956a241147b6f8813842c29a6e0c92ca9

comment:36 Changed 7 years ago by Dennis Mikkelson

RE 6901 MantidEV usability

Added controls on the first form to allow the user to restrict the range of Q values, and choose whether or not to do the Lorentz correction when converting to MD. This will be necessary for large molecule/protein crystallography.

refs #6901

Changeset: 0649a3e85ab59543ed9c3f33adf5abd1bdae127f

comment:37 Changed 7 years ago by Dennis Mikkelson

RE 6901 MantidEV usability

Added help menu item and linked it to initial, incomplete help page for MantidEV. The help page will be completed before the release.

refs #6901

Changeset: d410c9c49ed36c13d4c2c87f9748ebebfae715ad

comment:38 Changed 7 years ago by Dennis Mikkelson

RE 6901 MantidEV usability

When the user selects an event file to load, default names are now provided for the event, md and peaks workspaces.

refs #6901

Changeset: 9999ef74cc68a01e734ac8c71c8cca76a8db2a3c

comment:39 Changed 7 years ago by Dennis Mikkelson

RE 6901 MantidEV usability

Added option to append peaks to an existing peaks file, as requested by Xiaoping.

refs #6901

Changeset: f87f804a8fd81377dcc0754ab0f08cfc7a91ad60

comment:40 Changed 7 years ago by Dennis Mikkelson

RE 6901 MantidEV usability

Minor change to .ui file, to fix layout on some systems after adding (A-1) to appropriate labels on the Integrate tab.

refs #6901

Changeset: bf36730f214094da8544636eb6bd01859a0e5487

comment:41 Changed 7 years ago by Dennis Mikkelson

RE 6901 MantidEV usability

Changed |Q| range to go up to 35. Set MinRecursionDepth to 7 on ConvertToMD, to give at least 128x128x128 boxes. This makes the slice viewer a bit more usable.

Refs #6901

Changeset: b95c8d52d326a19f0a42d971164493ca1d88261c

comment:42 Changed 7 years ago by Dennis Mikkelson

RE 6901 MantidEV usability

Added units (A-1) to appropriate labels on the Integrate tab.

refs #6901

Changeset: 9d6131e1be668d96d3c1ca3d0ce2b9876da2e3f3

comment:43 Changed 7 years ago by Dennis Mikkelson

RE 6901 MantidEV usability improvements

Added menu option to reset default state of all GUI components. Removed some debug prints.

refs #6901

Changeset: 4fcf217956a241147b6f8813842c29a6e0c92ca9

comment:44 Changed 7 years ago by Dennis Mikkelson

RE 6901 MantidEV usability

Added controls on the first form to allow the user to restrict the range of Q values, and choose whether or not to do the Lorentz correction when converting to MD. This will be necessary for large molecule/protein crystallography.

refs #6901

Changeset: 0649a3e85ab59543ed9c3f33adf5abd1bdae127f

comment:45 Changed 7 years ago by Dennis Mikkelson

RE 6901 MantidEV usability

Added help menu item and linked it to initial, incomplete help page for MantidEV. The help page will be completed before the release.

refs #6901

Changeset: d410c9c49ed36c13d4c2c87f9748ebebfae715ad

comment:46 Changed 7 years ago by Dennis Mikkelson

RE 6901 MantidEV usability

When the user selects an event file to load, default names are now provided for the event, md and peaks workspaces.

refs #6901

Changeset: 9999ef74cc68a01e734ac8c71c8cca76a8db2a3c

comment:47 Changed 7 years ago by Dennis Mikkelson

RE 6901 MantidEV usability

Added option to append peaks to an existing peaks file, as requested by Xiaoping.

refs #6901

Changeset: f87f804a8fd81377dcc0754ab0f08cfc7a91ad60

comment:48 Changed 7 years ago by Dennis Mikkelson

RE 6901 MantidEV usability

Minor change to .ui file, to fix layout on some systems after adding (A-1) to appropriate labels on the Integrate tab.

refs #6901

Changeset: bf36730f214094da8544636eb6bd01859a0e5487

comment:49 Changed 7 years ago by Dennis Mikkelson

RE 6901 MantidEV usability

Changed |Q| range to go up to 35. Set MinRecursionDepth to 7 on ConvertToMD, to give at least 128x128x128 boxes. This makes the slice viewer a bit more usable.

Refs #6901

Changeset: b95c8d52d326a19f0a42d971164493ca1d88261c

comment:50 Changed 7 years ago by Dennis Mikkelson

RE 6901 MantidEV usability

Added units (A-1) to appropriate labels on the Integrate tab.

refs #6901

Changeset: 9d6131e1be668d96d3c1ca3d0ce2b9876da2e3f3

comment:51 Changed 7 years ago by Dennis Mikkelson

RE 6901 MantidEV usability improvements

Added menu option to reset default state of all GUI components. Removed some debug prints.

refs #6901

Changeset: 4fcf217956a241147b6f8813842c29a6e0c92ca9

comment:52 Changed 7 years ago by Dennis Mikkelson

RE 6901 MantidEV usability

Added controls on the first form to allow the user to restrict the range of Q values, and choose whether or not to do the Lorentz correction when converting to MD. This will be necessary for large molecule/protein crystallography.

refs #6901

Changeset: 0649a3e85ab59543ed9c3f33adf5abd1bdae127f

comment:53 Changed 7 years ago by Dennis Mikkelson

RE 6901 MantidEV usability

Added help menu item and linked it to initial, incomplete help page for MantidEV. The help page will be completed before the release.

refs #6901

Changeset: d410c9c49ed36c13d4c2c87f9748ebebfae715ad

comment:54 Changed 7 years ago by Dennis Mikkelson

RE 6901 MantidEV usability

When the user selects an event file to load, default names are now provided for the event, md and peaks workspaces.

refs #6901

Changeset: 9999ef74cc68a01e734ac8c71c8cca76a8db2a3c

comment:55 Changed 7 years ago by Dennis Mikkelson

RE 6901 MantidEV usability

Added option to append peaks to an existing peaks file, as requested by Xiaoping.

refs #6901

Changeset: f87f804a8fd81377dcc0754ab0f08cfc7a91ad60

comment:56 Changed 7 years ago by Dennis Mikkelson

RE 6901 MantidEV usability

Minor change to .ui file, to fix layout on some systems after adding (A-1) to appropriate labels on the Integrate tab.

refs #6901

Changeset: bf36730f214094da8544636eb6bd01859a0e5487

comment:57 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 7747

Note: See TracTickets for help on using tickets.