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

SANS: Throws bad alloc when running 2D reduce #8434

Closed
mantid-builder opened this issue Jul 30, 2013 · 3 comments
Closed

SANS: Throws bad alloc when running 2D reduce #8434

mantid-builder opened this issue Jul 30, 2013 · 3 comments
Assignees
Labels
SANS Issues and pull requests related to SANS

Comments

@mantid-builder
Copy link
Collaborator

sans_bugfix
This issue was originally TRAC 7589

Original Reporter: Samuel Jackson

SANS throws a bad alloc error when running a 2D reduce on a call to the Fit algorithm.

The following output is displayed:

Error in execution of algorithm Fit:
std::bad_alloc
MemoryError: 
  at line 35 in '`<Interface>`'
  caused by line 440 in '/opt/Mantid/scripts/SANS/ISISCommandInterface.py'
  caused by line 570 in '/opt/Mantid/scripts/SANS/ISISCommandInterface.py'
  caused by line 206 in '/opt/Mantid/bin/mantid/simpleapi.py'

Can be reproduced by following instructions at http://www.mantidproject.org/SANS_Data_Analysis_at_ISIS

@mantid-builder
Copy link
Collaborator Author

Gesner Passos (2013-07-30T11:58:56):
Input that cause the exception to be seen.


Gesner Passos (2013-07-30T12:07:43):
In reality, looking more into the details, the inputs were wrong. The range when performing REDUCE2D is from -0.1 to 0.1. So, when it tries to ask Fit to fit against an interval from 0.13 till 0.25 it fails.

The exception could be more graciously presented from Fit algorithm (checking that the limits are valid is a good sanity check), and could be avoided from the SANS interface as well.

But, the good news is that this but was not introduced in this release (reason for asking to be configure as blocking for release 2.6). We can deal with it later.


@NickDraper (2014-02-14T11:05:00):
Bulk move to assigned at the introduction of the triage step


@NickDraper (2014-03-07T16:20:30):
Ownership handed over to Peter

@mantid-builder
Copy link
Collaborator Author

http://trac.mantidproject.org/mantid/raw-attachment/ticket/7589/MaskFile.txt
(uploaded by Gesner Passos on 2013-07-30T11:58:56)


@mantid-builder mantid-builder added the SANS Issues and pull requests related to SANS label Jun 3, 2015
@Matthew-Andrew Matthew-Andrew added this to the Release 3.12 milestone Oct 30, 2017
@NickDraper NickDraper modified the milestones: Release 3.12, Release 3.13 Feb 27, 2018
@NickDraper NickDraper removed this from the Release 3.13 milestone Jul 9, 2018
@Matthew-Andrew
Copy link
Contributor

This is an issue with the old backend which is now deprecated.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
SANS Issues and pull requests related to SANS
Projects
None yet
Development

No branches or pull requests

5 participants