tommyod / KDEpy

Kernel Density Estimation in Python
https://kdepy.readthedocs.io/en/latest/
BSD 3-Clause "New" or "Revised" License
584 stars 90 forks source link

cutils compiles to package parent directory #150

Closed whitews closed 1 year ago

whitews commented 1 year ago

Hi Tommy,

Doing some testing I came across a few issues. One is that the compiled cutils object is stored in the directory above the package directory. You can see this when installing via pip by looking in the site-packages directory:

Screenshot 2023-09-17 at 7 06 45 PM

I can submit a PR with a fix if you'd like.

-Scott

tommyod commented 1 year ago

Another good catch. Please do submit a PR. I'll happily merge!

whitews commented 1 year ago

Great, I'll put this together. Also, I'll make separate PRs for the other issues after this one so it's easier to troubleshoot the Actions runners.