Ticket #2329 (closed: fixed)

Opened 10 years ago

Last modified 5 years ago

Load: Custom Dialog

Reported by: Nick Draper Owned by: Martyn Gigg
Priority: critical Milestone: Iteration 27
Component: Mantid Keywords:
Cc: Blocked By:
Blocking: Tester: Brett Tully

Description

Create a custom dialog for the Load Algorithm.

It will have two "fixed" parameters:

  • Filename
  • OutputWorkpace

Once filename is filled in and loses focus it should

  1. automatically fill in a possible name for OutputWorkspace (as LoadRawDialog)
  2. Call Load (Load will have to change to support this) to find out which underlying Loader it will use
  3. Dynamically create further controls for any other properties of the specific Load Algoritm (as the generic dialog does already)
  4. Redirect the ? help button to the correct help page

Finally update the menus & toolbars in the mantidplot GUI to remove the specific Load commands and just leave the general one. LoadDAE should be moved to only exist under the fine menu.

Change History

comment:1 Changed 10 years ago by Martyn Gigg

  • Status changed from new to accepted

comment:2 Changed 10 years ago by Martyn Gigg

(In [9071]) Update to the Load algorithm to allow running solely to discover the concrete loader type. Also updated workspace property to respect the optional flag in other cirumstances. Re #2329

comment:3 Changed 10 years ago by Martyn Gigg

(In [9080]) Some minor changes to neaten up the IDataFileChecker children and to move the static contants that were not attached to anything in particular. Re #2329

comment:4 Changed 10 years ago by Martyn Gigg

(In [9083]) Fix for static const initialization on windows? Re #2329

comment:5 Changed 10 years ago by Martyn Gigg

(In [9085]) Rubbish fix for Windows build until tomorrow when I figure out how to do it properly. Re #2329

comment:6 Changed 10 years ago by Martyn Gigg

(In [9102]) Proper fix for the static const declaration debacle in IDataFileChecker. The class was hard marked DLLExport rather than switching between the two depending on the IN_MANTID_API flag. This is the case in quite a lot if our code where DLLExport is used instead of the correct EXPORT_OPT_IN_{PACKAGE} (which I think need a more concise name for). Re #2329

comment:7 Changed 10 years ago by Martyn Gigg

(In [9113]) This should be everything I need for the Framework to make using the Load algorithm much simpler (including from C++). Re #2329

comment:8 Changed 10 years ago by Martyn Gigg

(In [9266]) Implemented a custom dialog for the Load algorithm. It now dynamically generates its properties from those that the concrete loaer has, Refs #2329. Also fixed a problem with loading grouped nexus files through the Load algorithm, Re #2362

comment:9 Changed 10 years ago by Martyn Gigg

(In [9283]) Fix for test problems with WorkspaceProperty types. Re #2329

comment:10 Changed 10 years ago by Martyn Gigg

(In [9284]) Fixed linking error on MantidPlot. Re #2329

comment:11 Changed 10 years ago by Martyn Gigg

(In [9316]) Modified LoadEventPreNeXus so that it is compatible with the Load algorithm, Refs #2168. Also made a couple of adjustments to the layout of the MWRunFiles widget to make it better at resizing with the dialogs it is attached to. Re #2329

comment:12 Changed 10 years ago by Martyn Gigg

(In [9320]) Fix the tests. Re #2329

comment:13 Changed 10 years ago by Martyn Gigg

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

comment:14 Changed 10 years ago by Brent Tully

  • Status changed from verify to verifying
  • Tester set to Brent Tully

comment:15 Changed 10 years ago by Brent Tully

  • Status changed from verifying to closed

Correct as of 1.1.9450

comment:16 Changed 10 years ago by Nick Draper

  • Tester changed from Brent Tully to Brett Tully

comment:17 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 3176

Note: See TracTickets for help on using tickets.