Opened 2 years ago
Last modified 2 weeks ago
#888 new defect
Support numpy v1.12.0+ - Array indices must be integers
Reported by: | krzywon | Owned by: | |
---|---|---|---|
Priority: | major | Milestone: | SasView 4.3.0 |
Component: | SasView | Keywords: | |
Cc: | Work Package: | SasView Bug Fixing |
Description
As of numpy v1.12.0, all array indices are required to be integers, not floating point. This is causing issues in unit tests in the Igor reader. As of now, Jenkins uses an older version of numpy, but going forward, we should try to use the latest versions of every package to minimize the number of specific packages needed.
Pull request where this was found:
https://github.com/SasView/sasview/pull/47
Travis CI build log where the errors are happening:
https://travis-ci.org/SasView/sasview/builds/212831209
Numpy changelog outlining this new feature:
https://github.com/numpy/numpy/blob/master/doc/release/1.12.0-notes.rst
Change History (5)
comment:1 Changed 2 years ago by krzywon
comment:2 Changed 2 years ago by krzywon
The following files have been fixed so far:
/sas/sascalc/dataloader/readers/IgorReader.py
/sas/sascalc/calculator/slit_length_calculator.py
A final run through is still required.
comment:3 Changed 18 months ago by krzywon
comment:4 Changed 17 months ago by butler
- Milestone changed from SasView 4.2.0 to SasView 4.3.0
The Igor Reader is confirmed to not use integer indices in numpy arrays, but there may be other places in sasview (and sasmodels) that have not been found yet.