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

BUG:linalg:interpolative: Crashing if k is requested too high in interp_decomp #20452

Open
ilayn opened this issue Apr 11, 2024 · 0 comments · May be fixed by #20558
Open

BUG:linalg:interpolative: Crashing if k is requested too high in interp_decomp #20452

ilayn opened this issue Apr 11, 2024 · 0 comments · May be fixed by #20558
Assignees
Labels
defect A clear bug or issue that prevents SciPy from being installed or used as expected Fortran Items related to the internal Fortran code base scipy.linalg
Milestone

Comments

@ilayn
Copy link
Member

ilayn commented Apr 11, 2024

Describe your issue.

I am rewriting linalg.interpolative Fortran code for #18367 and started discovering some edges. I don't know what the cause of this but probably some wrapper allocation issue. Here is just to record in time in case later we stumble upon it though I highly doubt anyone is using this. I'll close this with the rewrite PR.

Reproducing Code Example

import scipy.linalg.interpolative as lai
import numpy as np

A = np.random.rand(8, 6)
lai.interp_decomp(A, eps_or_k=10)

Error message

The kernel died, restarting...

SciPy/NumPy/Python version and system information

1.13.0 1.26.4 sys.version_info(major=3, minor=12, micro=1, releaselevel='final', serial=0)
Build Dependencies:
  blas:
    detection method: pkgconfig
    found: true
    include directory: /c/opt/64/include
    lib directory: /c/opt/64/lib
    name: openblas
    openblas configuration: USE_64BITINT= DYNAMIC_ARCH=1 DYNAMIC_OLDER= NO_CBLAS=
      NO_LAPACK= NO_LAPACKE= NO_AFFINITY=1 USE_OPENMP= ZEN MAX_THREADS=24
    pc file directory: c:/opt/64/lib/pkgconfig
    version: 0.3.26.dev
  lapack:
    detection method: pkgconfig
    found: true
    include directory: /c/opt/64/include
    lib directory: /c/opt/64/lib
    name: openblas
    openblas configuration: USE_64BITINT= DYNAMIC_ARCH=1 DYNAMIC_OLDER= NO_CBLAS=
      NO_LAPACK= NO_LAPACKE= NO_AFFINITY=1 USE_OPENMP= ZEN MAX_THREADS=24
    pc file directory: c:/opt/64/lib/pkgconfig
    version: 0.3.26.dev
  pybind11:
    detection method: config-tool
    include directory: unknown
    name: pybind11
    version: 2.12.0
Compilers:
  c:
    commands: cc
    linker: ld.bfd
    name: gcc
    version: 10.3.0
  c++:
    commands: c++
    linker: ld.bfd
    name: gcc
    version: 10.3.0
  cython:
    commands: cython
    linker: cython
    name: cython
    version: 3.0.10
  fortran:
    commands: gfortran
    linker: ld.bfd
    name: gcc
    version: 10.3.0
  pythran:
    include directory: C:\Users\runneradmin\AppData\Local\Temp\pip-build-env-tun37eh0\overlay\Lib\site-packages/pythran
    version: 0.15.0
Machine Information:
  build:
    cpu: x86_64
    endian: little
    family: x86_64
    system: windows
  cross-compiled: false
  host:
    cpu: x86_64
    endian: little
    family: x86_64
    system: windows
Python Information:
  path: C:\Users\runneradmin\AppData\Local\Temp\cibw-run-on03_hml\cp312-win_amd64\build\venv\Scripts\python.exe
  version: '3.12'
@ilayn ilayn added defect A clear bug or issue that prevents SciPy from being installed or used as expected scipy.linalg Fortran Items related to the internal Fortran code base labels Apr 11, 2024
@ilayn ilayn added this to the 1.14.0 milestone Apr 11, 2024
@ilayn ilayn self-assigned this Apr 11, 2024
@ilayn ilayn linked a pull request Apr 23, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
defect A clear bug or issue that prevents SciPy from being installed or used as expected Fortran Items related to the internal Fortran code base scipy.linalg
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants
@ilayn and others