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

refactor: use PushEvent in global openrank #1613

Merged

Conversation

frank-zsy
Copy link
Contributor

In this PR, I modify the global OpenRank algorithm for a little bit.

  • Use PushEvent instead of PullRequest merge event, because actually the pull request merge event will lead to a corresponding push event, but direct push will not contain a pull request event. Although we would like to encourage issue and pull request collaboration, direct pushes are also widely occurs in the open source world even in many really important projects. To fit the different collaboration model in open source world, use PushEvent with a relatively low weight will do.
  • Set the monthly activity limit for a single developer to the 75th percentile of the overall active users, which is approximately achieving 2 PRs per month to reach the limit for that month.
  • Output the users who active in more 300 repos within a month to help us to find out robot accounts.
  • If calculation for a month fails, we should restart the process because graph database may not stable.

@frank-zsy frank-zsy merged commit 1fb895b into X-lab2017:master Aug 31, 2024
1 check passed
@frank-zsy frank-zsy deleted the refactor/add-push-to-global-openrank branch August 31, 2024 13:10
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.

1 participant