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

analy 功能的标准模块实现 #74

Merged
merged 1 commit into from
Nov 12, 2018
Merged

analy 功能的标准模块实现 #74

merged 1 commit into from
Nov 12, 2018

Conversation

LuuMing
Copy link
Collaborator

@LuuMing LuuMing commented Nov 11, 2018

简单描述你的 PR 内容

初步实现 analy 的 python 标准模块,
函数说明:
get_analy(token,start,stop,owner,repo)

关联 issue

#22

说明

新建 analy 模块,__init__.py 中导入相关函数。core.py 中实现具体功能。
测试:

>>>result=grank.analy.get_analy(toke=XXXXXXXXXXXXX',owner='lctt',repo='grank',start='2017-01-01',stop='2018-11-08')
抓取数据:+++
开始活跃度分析:lctt/grank
分析 PR Commit Contributor
开始社区化分析:lctt/grank
>>>print(result)
>>>{'activity': 1.8390308796869257, 'social': 1.0}

增删部分

对 script 中的activity.py,crawler.py 中的 analy_repo 函数稍作修改。增加参数,使其可用作返回分析后的结果。

checklist

其他

希望后续中实现对提示信息的关闭

  • 我已经在本地执行了 pytest ,并通过了测试
  • 我已经为我新增的函数添加了测试
  • 我已经更新了 readme.md 中的相关代码
  • 我已经根据上方的描述检查了我的项目

@bestony
Copy link
Member

bestony commented Nov 11, 2018

记得 regenerate 你的token

@bestony bestony merged commit 7db753a into LCTT:master Nov 12, 2018
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