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

Update Python interface to fall back on ctypes.util.find_library #435

Closed
wants to merge 1 commit into from
Closed
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
9 changes: 4 additions & 5 deletions python/finufft/finufft/_finufft.py
Original file line number Diff line number Diff line change
Expand Up @@ -7,13 +7,11 @@
"""

import ctypes
import ctypes.util
import os
import warnings
import platform
import importlib.util

import numpy as np

from ctypes import c_double
from ctypes import c_int
from ctypes import c_float
Expand All @@ -26,13 +24,14 @@
c_double_p = ctypes.POINTER(c_double)
c_longlong_p = ctypes.POINTER(c_longlong)

# TODO: See if there is a way to improve this so it is less hacky.
lib = None
# Try to load a local library directly.
try:
lib = ctypes.cdll.LoadLibrary('libfinufft.so')
except OSError:
pass
libname = ctypes.util.find_library('finufft')
if libname is not None:
lib = ctypes.cdll.LoadLibrary(libname)

# Should that not work, try to find the full path of a packaged lib.
# The packaged lib should have a py/platform decorated name,
Expand Down