PaddlePaddle / PaddleHelix

Bio-Computing Platform Featuring Large-Scale Representation Learning and Multi-Task Deep Learning “螺旋桨”生物计算工具集
Other
1.02k stars 225 forks source link

Fix dependency name for scikit-learn in setup.py #283

Open linkedlist771 opened 6 months ago

linkedlist771 commented 6 months ago

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:

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.

CLAassistant commented 6 months ago

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

linkedlist771 commented 6 months ago

@xiaoyao4573 Can you review it?

Rex-HH commented 2 months ago

不太懂这个,现在这PaddleHelix 安装不了,有什么办法安装这个修改过的版本吗