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

pip install fails on latest version #35

Open
cdeimert opened this issue Jun 9, 2022 · 1 comment
Open

pip install fails on latest version #35

cdeimert opened this issue Jun 9, 2022 · 1 comment

Comments

@cdeimert
Copy link

cdeimert commented Jun 9, 2022

Hi,

I cloned the latest version of this package from github and tried installing it via pip install ., but the build failed with

...
cythoning psi_at_inf_cython.pyx to psi_at_inf_cython.c
error: C:\Users\deimertc\AppData\Local\Temp\pip-req-build-ynionnel\psi_at_inf_cython.pyx
----------------------------------------
ERROR: Failed building wheel for aestimo
...

I saw that setup.py is trying to cythonize the file "psi_at_inf_cython.pyx", but I didn't see this file anywhere and a search for "psi_at_inf" didn't show any references other than in setup.py. However, if I roll back to the v2.0.2 commit 3498e86, then psi_at_inf_cython.pyx appears in the main folder and pip install . works just fine. So I assume the issue is that psi_at_inf was removed in recent updates but setup.py is still trying to do something with it?

I'll likely want to play with the source code here, at minimum adding some new materials. Will I need to just branch off the v2.0.2 commit, or should the most recent version be workable?

Thanks,
Chris

@sblisesivdin
Copy link
Collaborator

sblisesivdin commented Jun 9, 2022

Yes. You are right. Last month I worked on the code and one of the change wa removing the cython support. I removed it because, normally 1d simulations did not take too much time. However, i still did not finish on my changes (like setup.py). I am too busy in these days. However without the need of setup py you can work with new aestimo. Just download, install numpy, scipy and matplotlib. Then you can run aestimo.py. If you add your path of aestimo.py to PATH variable of your system, you can it everywhere. Run aestimo.py -h for more. I hope i will work on setup.py soon (or you can submit your pull-request to change it. Thanks for using aestimo.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants