Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Unhandled exception when plotting live data from fake histogram instrument #36726

Open
rbauststfc opened this issue Jan 24, 2024 · 1 comment · May be fixed by #36775
Open

Unhandled exception when plotting live data from fake histogram instrument #36726

rbauststfc opened this issue Jan 24, 2024 · 1 comment · May be fixed by #36775
Assignees
Labels
Bug Issues and pull requests that are regressions or would be considered a bug by users (e.g. crashing) GUI Issues and pull requests specific to the Mantid Workbench GUI. ISIS Team: Core Issue and pull requests managed by the Core subteam at ISIS
Milestone

Comments

@rbauststfc
Copy link
Contributor

rbauststfc commented Jan 24, 2024

Describe the bug

Found during manual testing for the 6.9 release (#36702). Trying to open a plot on a workspace that is being updated with data from the fake ISIS_Histogram instrument causes an unhandled exception. In Mantid version 6.8 this would log an error message but not trigger the unhandled exception, so it appears to be a regression.

To Reproduce

Go through the following steps from the live data manual testing instructions to get the live data running:

  1. https://developer.mantidproject.org/Testing/LiveData/LiveDataTests.html#set-up-the-test-facility
  2. https://developer.mantidproject.org/Testing/LiveData/LiveDataTests.html#isis-histogram
  3. https://developer.mantidproject.org/Testing/LiveData/LiveDataTests.html#starting-a-live-data-session (just steps 1 and 2 here)
  4. https://developer.mantidproject.org/Testing/LiveData/LiveDataTests.html#live-histogram-data

Then, with live data running, right click on the workspace in the ADS and try to open a plot (select a single spectrum to plot). This should cause an unhandled exception.

Expected behavior

In previous versions an error was logged but it didn't cause the unhandled exception.

Platform/Version (please complete the following information):

  • Mantid Version - introduced since version 6.8.
@rbauststfc rbauststfc added the Bug Issues and pull requests that are regressions or would be considered a bug by users (e.g. crashing) label Jan 24, 2024
@rbauststfc rbauststfc added this to the Release 6.9 milestone Jan 24, 2024
@GuiMacielPereira GuiMacielPereira self-assigned this Jan 26, 2024
@thomashampson
Copy link
Contributor

For me this is throwing an unhandled exception in 6.8

@GuiMacielPereira GuiMacielPereira added the GUI Issues and pull requests specific to the Mantid Workbench GUI. label Feb 6, 2024
@sf1919 sf1919 modified the milestones: Release 6.9, Release 6.10 Feb 12, 2024
@sf1919 sf1919 added the ISIS Team: Core Issue and pull requests managed by the Core subteam at ISIS label Feb 20, 2024
@jhaigh0 jhaigh0 modified the milestones: Release 6.10, Release 6.11 May 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Issues and pull requests that are regressions or would be considered a bug by users (e.g. crashing) GUI Issues and pull requests specific to the Mantid Workbench GUI. ISIS Team: Core Issue and pull requests managed by the Core subteam at ISIS
Projects
Status: In progress
Status: Todo
Development

Successfully merging a pull request may close this issue.

5 participants