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

Pandas missing `pandas._libs.tslibs.timedeltas.so` #2978

Closed
jamesstidard opened this Issue Nov 7, 2017 · 11 comments

Comments

Projects
None yet
8 participants
@jamesstidard

jamesstidard commented Nov 7, 2017

I'm using pandas==0.21.0 as pyinstaller==3.3 and pyinstaller==3.4.dev'

When building with pyinstaller the /path/to/venv/lib/python3.4/site-packages/pandas/_libs/tslibs/timedeltas.cpython-34m.so is not copied over to the dist directory, like its over counterparts. Instead, I'm manually renaming and putting this into dist myself as dist/myapp/pandas._libs.tslibs.timedeltas.so.

Unsure which package is "at fault" for this or how I should go about resolving this. For example if this should be handled in a update of the project or I use something like the --add-binary flag to handle this case (assuming that's an appropriate usage of the command option).

UPDATE: --add-binary doesn't appear to help as the right-hand side of the binary_path:to_path doesn't appear to let you specifically the renamed binary and just uses the exact name from the package - in this case is incorrect. So I'm probably doing that wrong.

Thanks

@mdigi

This comment has been minimized.

mdigi commented Nov 7, 2017

I'm also having the same problem when upgrading from pandas 0.20.3 to 0.21.0.

@jamesstidard

This comment has been minimized.

jamesstidard commented Nov 7, 2017

Having a similar problem with scipy==1.0.0 path/to/venv/lib/python3.4/site-packages/scipy/_lib/messagestream.cpython-34m.so.

This also needs renaming, which is okay o do in my bash script after with mv. However this solution wont help for --onefile - which where I'm hoping to get to.

@myeol2

This comment has been minimized.

myeol2 commented Nov 13, 2017

Having same problem with @jamesstidard , and can solve this problem thanks to his advice.

But still i have same problem with --onefile !

@lneuhaus

This comment has been minimized.

Contributor

lneuhaus commented Nov 13, 2017

+1 having the same problem as @jamesstidard

lneuhaus pushed a commit to lneuhaus/pyinstaller that referenced this issue Nov 13, 2017

lneuhaus added a commit to lneuhaus/pyinstaller that referenced this issue Nov 14, 2017

Hooks: Add hook-pandas.py to fix issue pyinstaller#2978.
pandas>=0.21 has a hidden import of "pandas._libs.tslibs.timedeltas.so".
@lneuhaus

This comment has been minimized.

Contributor

lneuhaus commented Nov 14, 2017

I added a pull-request that fixes this issue: #2998

Tested with pandas 0.21 and python 3.5 on a linux machine (travis). Not sure if my way to fix it is the proper one though.

@jamesstidard

This comment has been minimized.

jamesstidard commented Nov 14, 2017

Very much appreciated @lneuhaus , thanks. I'm using hidden imports for some other library I'm using but didn't think to try it with a *.so. I guess these are c compiled libraries with one of the python bindings - still not to familier with that. Thanks again

@lneuhaus

This comment has been minimized.

Contributor

lneuhaus commented Nov 14, 2017

@jamesstidard it was your comment that pointed me to this issue so all the thanks goes back to you i guess :)

@su79eu7k

This comment has been minimized.

su79eu7k commented Nov 14, 2017

Thank you @lneuhaus your solution worked in Windows also. (Windows / Python3.5 / Pandas 0.21.0 / Conda env)

More detail about my problem #2999

@htgoebel htgoebel added the hooks label Dec 6, 2017

@joshjacobson

This comment has been minimized.

joshjacobson commented Feb 16, 2018

@lneuhaus This is not working on MacOS (10.13.2) for me. Per the pull request, I created that hook file and included it in the PyInstaller/Hooks/ directory.

@htgoebel

This comment has been minimized.

Member

htgoebel commented Mar 13, 2018

Pull-request merged.

@arnabsinha99

This comment has been minimized.

arnabsinha99 commented Aug 27, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment