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

Import of ensmallen fails due to missing imports #192

Open
caufieldjh opened this issue Dec 6, 2022 · 2 comments
Open

Import of ensmallen fails due to missing imports #192

caufieldjh opened this issue Dec 6, 2022 · 2 comments

Comments

@caufieldjh
Copy link
Contributor

Some (or all?) dependencies aren't getting installed by pip.

Example:
The cpuinfo module may not get installed by pip, but it's imported in the __init__.py.
On a fresh pip install (from grape) and venv:

>>> from ensmallen import Graph
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/harry/test85812/lib/python3.8/site-packages/ensmallen/__init__.py", line 8, in <module>
    import cpuinfo
ModuleNotFoundError: No module named 'cpuinfo'

I see py-cpuinfo is in the Cargo.toml, and if I install it with pip then this solves the problem, but...
then I get a ModuleNotFoundError for bioregistry.
Same for downloaders.

Ensmallen is 0.8.35.

@LucaCappelletti94
Copy link
Member

We have just updated the build using the newer maturin version, possibly it has some issues. I will look into it.

@LucaCappelletti94
Copy link
Member

The issue continues to be present ensmallen-wise. I have patched it by adding the requirements to grape.

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