Ticket #11059 (closed: fixed)
StartLiveData ILiveListener Properties
Reported by: | Owen Arnold | Owned by: | Roman Tolchenov |
---|---|---|---|
Priority: | critical | Milestone: | Release 3.4 |
Component: | Framework | Keywords: | PatchCandidate |
Cc: | Blocked By: | ||
Blocking: | Tester: | Federico M Pouzols |
Description
1) The documentation for http://docs.mantidproject.org/nightly/algorithms/StartLiveData-v1.html is misleading, because the properties that get added from the ILiveListener after the instrument has been set, do not appear in the help. SpectraList for example from the histogram ILiveListener.
2) Setting the ILiveListener properties in python may not be possible via the usual functional syntax in python, because the ILiveListener properties are not added to StartLiveData until setProperty("Instrument", "...") is called.
This is a worthy patch candidate, as users will currently find it hard to use live data processing via a script in these circumstances.
Change History
comment:2 Changed 6 years ago by Roman Tolchenov
- Status changed from assigned to inprogress
Re #11059. Added ISIS-specific properties to StartLiveData.
Changeset: eeea9bad4973c009e3ce0e371fa826380a95fd35
comment:3 Changed 6 years ago by Roman Tolchenov
- Status changed from inprogress to verify
- Resolution set to fixed
This is being verified as pull request #234.
comment:4 Changed 6 years ago by Federico M Pouzols
- Status changed from verify to verifying
- Tester set to Federico M Pouzols
comment:6 Changed 6 years ago by Roman Tolchenov
Re #11059. Fix a typo in example code.
Changeset: d26a4d599e74a6873b42c98e0b247c1ab00e8dea
comment:7 Changed 6 years ago by Federico M Pouzols
All looks good, and tests pass, including the new doc test.
comment:8 Changed 6 years ago by Federico M Pouzols
- Status changed from verifying to closed
Merge pull request #234 from mantidproject/11059_ilivelistener_properties
Added ISIS-specific properties to StartLiveData.
Full changeset: ee87e1550d9ddff27878f77320001686ba323ba1
comment:9 Changed 5 years ago by Nick Draper
Somehow these slipped through without a resolution. Set to Fixed.
comment:10 Changed 5 years ago by Stuart Campbell
This ticket has been transferred to github issue 11898