Ticket #11059 (closed: fixed)

Opened 6 years ago

Last modified 5 years ago

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:1 Changed 6 years ago by Owen Arnold

  • Status changed from new to assigned

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:5 Changed 6 years ago by Roman Tolchenov

Yes, I'll fix that

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

Note: See TracTickets for help on using tickets.