Ticket #4669 (closed: fixed)

Opened 9 years ago

Last modified 5 years ago

Algorithm*Dialog() commands do not work right in MantidPlot

Reported by: Janik Zikovsky Owned by: Martyn Gigg
Priority: critical Milestone: Release 2.0
Component: Mantid Keywords:
Cc: Blocked By:
Blocking: Tester: Owen Arnold

Description

The command: 

LoadEventNexusDialog("output", "filename", True, 100, 10)

opens the LoadEventNexus Dialog but does not fill in any of the values. Isn't it supposed to? It works if you use the keyword argument.

This is with the new python API enabled.

Change History

comment:1 Changed 9 years ago by Martyn Gigg

  • Status changed from new to accepted

comment:2 Changed 9 years ago by Martyn Gigg

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

comment:3 Changed 9 years ago by Martyn Gigg

Refs #4669. Fix dialog functions using positional arguments.

Changeset: 9ebf3145537bdf0bfdcb14f59b72ea37d20a34e2

comment:4 Changed 9 years ago by Michael Reuter

  • Status changed from verify to verifying
  • Tester set to Michael Reuter

comment:5 Changed 9 years ago by Michael Reuter

  • Status changed from verifying to reopened
  • Resolution fixed deleted

This seems to work for string and boolean arguments. However, none of the number arguments show up. Also, are the default arguments in the dialog supposed to come up disabled?

comment:6 Changed 9 years ago by Martyn Gigg

So it turns out that the number arguments actually correspond to parameters that are hidden by default. If you tick LoadMonitors and MonitorsAsEvents then they show up. So actually this needs LoadMonitors=True, MonitorsAsEvents=True to actually make sense.

The defaults that are set by this command are supposed to disable the input boxes. They can be left enabled by having an additional keyword Enabled='PropertyName'.

comment:7 Changed 9 years ago by Martyn Gigg

  • Status changed from reopened to accepted

comment:8 Changed 9 years ago by Martyn Gigg

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

comment:9 Changed 9 years ago by Owen Arnold

  • Status changed from verify to verifying
  • Tester changed from Michael Reuter to Owen Arnold

comment:10 Changed 9 years ago by Owen Arnold

  • Status changed from verifying to closed

comment:11 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​5516

Note: See TracTickets for help on using tickets.