Skip to content

Comments

Package up#157

Merged
dataabc merged 2 commits intodataabc:masterfrom
songzy12:master
May 29, 2020
Merged

Package up#157
dataabc merged 2 commits intodataabc:masterfrom
songzy12:master

Conversation

@songzy12
Copy link
Collaborator

Related: #156

@dataabc
Copy link
Owner

dataabc commented May 29, 2020

感谢共享代码。

我在本地电脑(win10)和服务器(Ubuntu)上测试了下代码。两者的共同点是:都必需在weibo_spider的安装目录下配置config.json,不能自定义config.json的位置。其实这不算问题,因为我们可以在后面完善。我现在纠结的是开源协议。

本项目的目的是为个人使用者服务的,方便他们使用。他们使用本项目的目的可能是为毕设收集资料,或者为机器学习或自然语言处理收集语料,或者其它研究,都是非盈利性质的,这也算是项目的初衷。我发现BSD License是允许商用的,这是我最担心的一点。因为本程序是在获取微博用户的数据,这些数据的所有权应该属于微博和发布者,我没有资格授权。如果别人把它用在盈利的项目上可能会有风险,因此应该使用比BSD更严格的协议。

@songzy12
Copy link
Collaborator Author

  1. config.json 的位置我可以来试着改一下,当时只测了一下能调用后没有仔细测试。
  2. 协议的部分有道理的,当前的协议我是参考的 flask 的协议随便添加的。作者可以看一下哪个协议比较适合,我可以顺便把这部分也修改一下。

@dataabc
Copy link
Owner

dataabc commented May 29, 2020

想了想,发现不添加开源协议是最严格的。没有发现严格禁止商用的协议,如果没有合适的协议,可以暂时不写,以后再添加也可以

@songzy12
Copy link
Collaborator Author

现在可以通过这种方式来指定任意 config.json 路径了:

python3 -m weibo_spider --config_path="xxxx/config.json"

@dataabc dataabc merged commit 5734620 into dataabc:master May 29, 2020
@dataabc
Copy link
Owner

dataabc commented May 29, 2020

已merge,再次感谢,感谢不辞劳苦的修改代码和为开源做的贡献

@songzy12
Copy link
Collaborator Author

作者客气了。

当前 README 里的使用方法可能还需要跟着修改一下。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants