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

Cannot load ISIS event data file #37213

Closed
mducle opened this issue Apr 24, 2024 · 2 comments
Closed

Cannot load ISIS event data file #37213

mducle opened this issue Apr 24, 2024 · 2 comments
Assignees
Labels
Bug Issues and pull requests that are regressions or would be considered a bug by users (e.g. crashing) ISIS Team: Core Issue and pull requests managed by the Core subteam at ISIS

Comments

@mducle
Copy link
Member

mducle commented Apr 24, 2024

Original reporter: ISIS / M. Aouane

Describe the bug

Mantid gives an error:

Error in execution of algorithm LoadEventNexus:
Cannot guess ending value from a TimeSeriesProperty that contains only a single time

when trying to load this particular file:

from mantid.simpleapi import *
Load(Filename='/data/instrument/MARI/CYCLE20241/RB0/MAR29527.n001', OutputWorkspace='MAR29527')

The file should be publicly accessible on the DAaaS system - or I can send directly (it's a bit big so I don't want to attach it to this issue).

To Reproduce

Run the script above

Expected behavior

The file loads without error.

Screenshots

Platform/Version (please complete the following information):

  • OS: Rocky 8 Linux
  • OS Version:
  • Mantid Version: 6.9.1 (also checked on 6.7 and 6.8 which give the same error).

Additional context

The file loads fine with nexpy and hdfview so is a valid NeXus file. The error seems to be in the RoI (region of interest) processing in LoadEventNexus which in turn calls TimeROI::replaceROI which gives the error, but I can't figure out the bug from just glancing at the code - all of the log properties in the file have more than one entry... so it seems something wrong in the logic in LoadNexusEvent.

@mducle mducle added the Bug Issues and pull requests that are regressions or would be considered a bug by users (e.g. crashing) label Apr 24, 2024
@sf1919 sf1919 added the ISIS Team: Core Issue and pull requests managed by the Core subteam at ISIS label Apr 30, 2024
@mducle
Copy link
Member Author

mducle commented Apr 30, 2024

Fixed in nightly 6.9.1.20240425.1854.

@mducle mducle closed this as completed Apr 30, 2024
@GuiMacielPereira
Copy link
Contributor

Reopening just to add that I suspect it was the work on #37141 that unintentionally fixed it

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) ISIS Team: Core Issue and pull requests managed by the Core subteam at ISIS
Projects
Status: Done
Development

No branches or pull requests

3 participants