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

Feature Request: extract sdk/api into a seperat package #112

Open
xyb opened this issue Mar 14, 2023 · 3 comments
Open

Feature Request: extract sdk/api into a seperat package #112

xyb opened this issue Mar 14, 2023 · 3 comments
Labels
enhancement New feature or request

Comments

@xyb
Copy link

xyb commented Mar 14, 2023

我最近想做一个基于 BaiduPCS-Py 的 web 服务,它只需要用到封装的 sdk,client 的代码以及依赖是不需要的。如果能够把 sdk/api 剥离成一个独立的包,会更轻量一些,第三方扩展起来更方便。

@PeterDing
Copy link
Owner

"轻量" 是指依赖更少的库吗?如果只是如此,我认为分包是没有必要的。分包不会有功能、易用性、性能的提升。

@xyb
Copy link
Author

xyb commented Mar 15, 2023

轻量是指 api 代码只需要依赖 requests 就可以了,其他的库都不是强制安装的。比如,使用 api 的人可能用自己的下载库,那就不需要装 aget;也可能不需要加密功能,就不需要安装加密库 simple_cipher 和 cython。现在这些都是用 BaiduPCS-Py 写 app 的开发者无法选择的,因为 pip install BaiduPCS-Py 就必须安装它们,而这些库要么比较大,要么要准备编译库,会带来额外的麻烦,这就很不“轻量”了。

@PeterDing PeterDing added the enhancement New feature or request label Mar 15, 2023
@PeterDing
Copy link
Owner

感谢反馈,这是有用的建议。后续我考虑怎么把包的依赖拆分为可选项。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants