PaddleHelix icon indicating copy to clipboard operation
PaddleHelix copied to clipboard

Fix dependency name for scikit-learn in setup.py

Open linkedlist771 opened this issue 1 month ago • 2 comments

Description:

This pull request fixes an issue in setup.py where the scikit-learn dependency was incorrectly listed as "sklearn" instead of the correct name "scikit-learn". This change ensures that the correct package is installed when setting up the project.

Changes Made:

  • Modified line 119 in setup.py:
    • Replaced "sklearn" with "scikit-learn" in the requires list.

Reason for Change:

The current dependency name "sklearn" is incorrect and will lead to failure when installing project dependencies. The correct package name for scikit-learn is "scikit-learn". "sklearn" has been deprecated, if we use this name, a runtime Error will raise(My env, ubuntu 20.04). This change ensures that the setup process installs the correct package.

Test Coverage:

This change does not affect the project's functionality and does not require additional testing.

Additional Notes:

None.

linkedlist771 avatar May 23 '24 06:05 linkedlist771