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

Missing dependency for scikit-learn #59

Closed
p3dda opened this issue Dec 10, 2023 · 1 comment
Closed

Missing dependency for scikit-learn #59

p3dda opened this issue Dec 10, 2023 · 1 comment
Labels
type: bug Something isn't working

Comments

@p3dda
Copy link

p3dda commented Dec 10, 2023

There seems to be a dependency missing in pyproject.tomp.

After installation from git on a clean linux environment (python 3.11), I get following traceback:

Traceback (most recent call last):
  File "<string>", line 1, in <module>
  File "/usr/local/lib/python3.11/importlib/__init__.py", line 126, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "<frozen importlib._bootstrap>", line 1204, in _gcd_import
  File "<frozen importlib._bootstrap>", line 1176, in _find_and_load
  File "<frozen importlib._bootstrap>", line 1147, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 690, in _load_unlocked
  File "<frozen importlib._bootstrap_external>", line 940, in exec_module
  File "<frozen importlib._bootstrap>", line 241, in _call_with_frames_removed
  File "/src/geo_activity_playground/__main__.py", line 11, in <module>
    from geo_activity_playground.heatmap import generate_heatmaps_per_cluster
  File "/src/geo_activity_playground/heatmap.py", line 7, in <module>
    import sklearn.cluster
ModuleNotFoundError: No module named 'sklearn'

Neither scikit-learn nor sklearn (which is deprecated anyways) is listed in the pyproject.toml

@martin-ueding martin-ueding added the type: bug Something isn't working label Dec 10, 2023
@martin-ueding
Copy link
Owner

Thanks for the report! I was too eager to remove it after changing the way that I compute the clusters for the explorer tiles. I've added again on master.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants