Closed gemmaguest closed 1 year ago
Changing the estimate from 2 to 5 because I think there are probably quite a few things that will need fixing up to get this working, especially regarding adding a new instrument if that is required.
This issue has been automatically marked as stale because it has not had activity in 6 months. It will be closed in 7 days if no further activity occurs. Allowing issues to close as stale helps us filter out issues which can wait for future development time. All issues closed by stale bot act like normal issues; they can be searched for, commented on or reopened at any point. If you'd like a closed stale issue to be considered, feel free to either re-open the issue directly or contact a developer. To extend the lifetime of an issue please comment below, it helps us see that this is still affecting you and you want it fixed in the near-future. Extending the lifetime of an issue may cause the development team to prioritise it over other issues, which may be closed as stale instead.
This issue has been closed automatically. If this still affects you please re-open this issue with a comment or contact us so we can look into resolving it.
The ISIS reflectometry GUI has a button to start the live data monitor and to perform a standard reflectometry reduction on it using the settings in the GUI. Currently however it is not possible to perform slicing of event data using the settings on the Event Handling tab - this will fail because the live data is always loaded in histo mode.
To do this we will need to:
ReflectometryReductionOneLiveData
can be run as post-processing on live event data. I suspect there may be assumptions in the reduction that might complicate thisRunsPresenter::setupLiveDataMonitorAlgorithm
to use event settings if event handling is enabled on the GUI.OFFSPEC_EVENT
can be run in live data event mode. This is not in the list of supported instruments for the GUI so it will need to be added; I'm not sure whether we want it to appear in the drop-down box though because that would be confusing (check with scientists), so it might require some re-work to allow that to be used but hidden. We might also hit some other problems with adding a "new" instrument.