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

Add the dataset of PR review comments #116

Merged
merged 2 commits into from
Mar 25, 2019

Conversation

warenlg
Copy link
Contributor

@warenlg warenlg commented Mar 22, 2019

Signed-off-by: Waren Long waren@sourced.tech

ReviewComments/README.md Outdated Show resolved Hide resolved
ReviewComments/README.md Outdated Show resolved Hide resolved
ReviewComments/README.md Outdated Show resolved Hide resolved
ReviewComments/README.md Outdated Show resolved Hide resolved
ReviewComments/README.md Show resolved Hide resolved
@vmarkovtsev
Copy link
Collaborator

Besides, @warenlg you should post here all the scripts/notebooks you used to generate this dataset. No need to polish them, they are just for the reference.

@warenlg warenlg force-pushed the PR-review-comments branch from 9db6a6b to 0f605bb Compare March 22, 2019 10:15
Copy link
Member

@marnovo marnovo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@warenlg agreed w/ Vadim's points.

Also please add the dataset to the root README.md index akin to the others we have so it's more obvious to users it exists.

Signed-off-by: Waren Long <waren@sourced.tech>
@warenlg warenlg force-pushed the PR-review-comments branch 2 times, most recently from 9a0cc9e to ae16318 Compare March 25, 2019 15:37
@warenlg
Copy link
Contributor Author

warenlg commented Mar 25, 2019

Done, I added 2 notebooks:

  1. to generate the dataset
  2. to process it and show some statistics

@warenlg warenlg force-pushed the PR-review-comments branch from ae16318 to 244402b Compare March 25, 2019 15:47
ReviewComments/README.md Outdated Show resolved Hide resolved
ReviewComments/README.md Outdated Show resolved Hide resolved
Signed-off-by: Waren Long <waren@sourced.tech>
@warenlg warenlg force-pushed the PR-review-comments branch from 244402b to 3813272 Compare March 25, 2019 17:00
@vmarkovtsev
Copy link
Collaborator

vmarkovtsev commented Mar 25, 2019

@marnovo Taking the liberty of merging this one. Waren has applied your suggestion with the top-level README mention.

@vmarkovtsev vmarkovtsev merged commit 79a319f into src-d:master Mar 25, 2019
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.

3 participants