HaasCP / mocca

Deprecated open-source Python project to analyze HPLC–DAD raw data.
MIT License
40 stars 12 forks source link

Install fails: The 'sklearn' PyPI package is deprecated, use 'scikit-learn' #1

Closed nhadler closed 1 year ago

nhadler commented 1 year ago

When attempting to install the mocca package, it fails due to the sklearn being depreciated, and should be replaced with scikit-learn. I was able to install by setting the environmental variable, but this isn't ideal.

  error: subprocess-exited-with-error

  × python setup.py egg_info did not run successfully.
  │ exit code: 1
  ╰─> [18 lines of output]
      The 'sklearn' PyPI package is deprecated, use 'scikit-learn'
      rather than 'sklearn' for pip commands.

      Here is how to fix this error in the main use cases:
      - use 'pip install scikit-learn' rather than 'pip install sklearn'
      - replace 'sklearn' by 'scikit-learn' in your pip requirements files
        (requirements.txt, setup.py, setup.cfg, Pipfile, etc ...)
      - if the 'sklearn' package is used by one of your dependencies,
        it would be great if you take some time to track which package uses
        'sklearn' instead of 'scikit-learn' and report it to their issue tracker
      - as a last resort, set the environment variable
        SKLEARN_ALLOW_DEPRECATED_SKLEARN_PACKAGE_INSTALL=True to avoid this error

      More information is available at
      https://github.com/scikit-learn/sklearn-pypi-package

      If the previous advice does not cover your use case, feel free to report it at
      https://github.com/scikit-learn/sklearn-pypi-package/issues/new
      [end of output]
HaasCP commented 1 year ago

Works fine for me for the installation procedure described in the docs using Python 3.9. I cannot reproduce the error. In case, you get the error following the procedure described in the docs, feel free to share more details on your environment.

I hope this helps. Have a great Sunday!

HaasCP commented 2 months ago

Sorry for not having checked this issue in detail earlier. I have fixed the dependency now.

This repo will be deprecated now, for a supported version see the link in the readme file.