Closed peterfpeterson closed 1 year ago
From doing a full run locally the following doctests are failing for me (someone should verify):
algorithms/LoadEMU-v1.rst is for ORNL, @peterfpeterson do you know if it should include the monitors?
@AnthonyLim23 EMU is an instrument at ANSTO
@geishm-ansto should LoadEMU include monitors?
@AnthonyLim23 EMU did not include a beam monitor. It is a backscattering instrument with a doppler shift and the scientist relied on the direct measurements as a pseudo-monitor for flux normalisation.
@geishm-ansto so does the algorithm need changing to not include the monitors?
@geishm-ansto The instrument definition includes this extra bank, which are loaded in as monitors. Is this new/expected?
It looks like the monitors were added to the IDF 3 months or so ago, which is the reason for the additional spectra appearing in the data files on loading. With that being the case, I'm going to update the doc test to match this as it's the current Mantid behaviour. If the monitors shouldn't be there, they can be removed from the IDF and this doctest can be updated alongside.
@ConorMFinn The history with the EMU instrument is that the (pseudo beam monitor) tube was always there but without electronics connected so there was no data and the original IDF did not include the spare tube. Now that it is active the IDF was updated. The early data files could be loaded with the new IDF but have zero events recorded for that tube. When the change was submitted the doc-tests passed at the time (?). Your change sounds fine, thanks.
After the following doc-tests are fixed, running them should be re-enabled on the osx build.
To coordinate work, please add your name/facility to items when work is started.
From doing a full run locally the following doctests are failing for me (someone should verify):
Todo
algorithms/SaveVulcanGSS-v1.rstAlgorithm hard deprecated, usage code removed.Not erroring (for @ConorMFinn on Ubuntu 18.04)