Ticket #4669 (closed: fixed)
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: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: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:11 Changed 5 years ago by Stuart Campbell
This ticket has been transferred to github issue 5516