Peer Review History

Original SubmissionJuly 17, 2019
Decision Letter - Tiago P. Peixoto, Editor

PONE-D-19-20119

A Network-Centric Approach for Estimating Trust between Open Source Software Developers

PLOS ONE

Dear Dr Murukannaiah,

Thank you for submitting your manuscript to PLOS ONE. After careful consideration, we feel that it has merit but does not fully meet PLOS ONE’s publication criteria as it currently stands. Therefore, we invite you to submit a revised version of the manuscript that addresses the points raised during the review process.

Please refer to the attached reviewer's comment and address them in detail in your re-submission.

We would appreciate receiving your revised manuscript by Oct 17 2019 11:59PM. When you are ready to submit your revision, log on to https://www.editorialmanager.com/pone/ and select the 'Submissions Needing Revision' folder to locate your manuscript file.

If you would like to make changes to your financial disclosure, please include your updated statement in your cover letter.

To enhance the reproducibility of your results, we recommend that if applicable you deposit your laboratory protocols in protocols.io, where a protocol can be assigned its own identifier (DOI) such that it can be cited independently in the future. For instructions see: http://journals.plos.org/plosone/s/submission-guidelines#loc-laboratory-protocols

Please include the following items when submitting your revised manuscript:

  • A rebuttal letter that responds to each point raised by the academic editor and reviewer(s). This letter should be uploaded as separate file and labeled 'Response to Reviewers'.
  • A marked-up copy of your manuscript that highlights changes made to the original version. This file should be uploaded as separate file and labeled 'Revised Manuscript with Track Changes'.
  • An unmarked version of your revised paper without tracked changes. This file should be uploaded as separate file and labeled 'Manuscript'.

Please note while forming your response, if your article is accepted, you may have the opportunity to make the peer review history publicly available. The record will include editor decision letters (with reviews) and your responses to reviewer comments. If eligible, we will contact you to opt in or out.

We look forward to receiving your revised manuscript.

Kind regards,

Tiago P. Peixoto

Academic Editor

PLOS ONE

Journal Requirements:

When submitting your revision, we need you to address these additional requirements.

Please ensure that your manuscript meets PLOS ONE's style requirements, including those for file naming. The PLOS ONE style templates can be found at

http://www.journals.plos.org/plosone/s/file?id=wjVg/PLOSOne_formatting_sample_main_body.pdf and http://www.journals.plos.org/plosone/s/file?id=ba62/PLOSOne_formatting_sample_title_authors_affiliations.pdf

1. In your Methods section, please include additional information about your dataset and ensure that you have included a statement specifying whether the collection method complied with the terms and conditions for the website.

[Note: HTML markup is below. Please do not edit.]

Reviewers' comments:

Reviewer's Responses to Questions

Comments to the Author

1. Is the manuscript technically sound, and do the data support the conclusions?

The manuscript must describe a technically sound piece of scientific research with data that supports the conclusions. Experiments must have been conducted rigorously, with appropriate controls, replication, and sample sizes. The conclusions must be drawn appropriately based on the data presented.

Reviewer #1: Partly

**********

2. Has the statistical analysis been performed appropriately and rigorously?

Reviewer #1: Yes

**********

3. Have the authors made all data underlying the findings in their manuscript fully available?

The PLOS Data policy requires authors to make all data underlying the findings described in their manuscript fully available without restriction, with rare exception (please refer to the Data Availability Statement in the manuscript PDF file). The data should be provided as part of the manuscript or its supporting information, or deposited to a public repository. For example, in addition to summary statistics, the data points behind means, medians and variance measures should be available. If there are restrictions on publicly sharing data—e.g. participant privacy or use of data from a third party—those must be specified.

Reviewer #1: Yes

**********

4. Is the manuscript presented in an intelligible fashion and written in standard English?

PLOS ONE does not copyedit accepted manuscripts, so the language in submitted articles must be clear, correct, and unambiguous. Any typographical or grammatical errors should be corrected at revision, so please note any specific errors here.

Reviewer #1: Yes

**********

5. Review Comments to the Author

Please use the space provided to explain your answers to the questions above. You may also include additional comments for the author, including concerns about dual publication, research ethics, or publication ethics. (Please upload your review as an attachment if it exceeds 20,000 characters)

Reviewer #1: Summary

-------

The authors describe a method for representing trust relationships within GitHub projects using pull requests. They develop a labeled dataset of comments indicating "trusted language" (or lack thereof), and test their methodology on this dataset. They further use this method to create a classifier for pull request acceptance or rejection, with a comparison to a simple baseline.

The authors frame their work in the context of prior work regarding trust, and motivate it in a sound manner. In addition, the paper is very well written and easy to read and follow.

There are a number of revisions I would like to see made and / or discussed by the authors before I recommend publication.

Review

-------

* In general, there are a few spots in which some conclusions are drawn that, I'd argue, are not fully supported by the data; an example is outlined below (line 428). As the primary consideration is experimental robustness and not "practical applicability", I would suggest toning back some of the stronger results statements. Most statements are acceptable and the subtlety is appreciated (e.g., lines 407-412 and lines 418-419).

* Line 43: How is a "top" Python project defined?

* Line 92-95: This description is somewhat confusing. How does each parameter take a value in [0, 1], yet three parameters add up to 1? I believe this is better explained later in the paper, but some small introduction or explanation here would make things a bit easier to follow.

* Line 209-210: To my knowledge, there is no precedent that the mean of word vectors in a document is representative of the content as a whole; only that word vectors are pair-wise close to each other in embedding space as measured by cosine similarity (and in compositions of cosine similarity). I would suggest citing a reference to such work, or explicitly stating this as a threat to validity.

* Section 3.2.3: It is unclear at what level the observations (i.e., "rows") are for the regression. Are they per-comment? Per pull request? Later (in results) this becomes clear, but stating it here may be a good idea.

* Fig. 2 / Section 3.2.3: What type of cross-validation was performed? K-fold (with what value of k)? Would results change if cross-validation was performed with respect to time? Contribution guidelines and the expected quality of accepted pull requests may change drastically over time, and may affect underlying trust relationships. Would results change if cross-validation was performed across repositories? Projects have different views of acceptable pull request quality / submission behavior, which will likely affect trust relationships.

* Line 329-331: Re (2): How are auto-generated comments detected? Re (4): What precisely constitutes "referring to the pull requester"? The usage of an @-mention directed towards the pull requester? Or some other signaling mechanism?

* Line 333: How was this replacement performed? Is a list of mappings between abbreviations and their full forms available?

* Section 4.2 / 5.1: Interpreting performance based on MAE without a clear baseline is difficult. What is the MAE in the worst case scenario? E.g., when always assigning a 5 when the true value is a 1, and analogously assigning maximum "worst possible values" to the test data. What would the MAE be if one, e.g., randomly assigned labels (uniformly from 1 to 5) to the test set? A table or figure representing the distribution of labels in the train and test set would also be helpful here. In addition, I would also appreciate a variance-based (or at least "variance informed") evaluation metric in addition to MAE.

Was classification, as opposed to regression, also considered here? I understand that the regression approach preserves distance between classes defined on a Likert (or Likert-like) scale, but I am curious as to whether the authors believe a classification approach would drastically change their results.

* Section 4.3 / 5.2.2: I very much appreciate the usage of a simple baseline model for comparison. Is there a reason that you did not include features used in prior work (such as in the cited work by Tsay et al. [14], who build a model for pull request acceptance as done in this work)?

* Section 5: As stated above, it is unclear what method of cross-validation is performed. Please clarify.

* Table 5: I assume this performance is calculated on the test set, but an explicit statement of such in the table caption would be useful.

* Line 381-388: It is stated that trust is computed in the CDN described by D_train. Is this trust value then considered "fixed" and used in the corresponding models trained on 70% of data from D_test? A bit of additional explicit description and clarification could be added here.

* Line 428: The statement that Fig 4 provides "strong evidence" for the given conclusion is a stretch; Cliff's delta is either small or negligible in all cases. Though journal guidelines dictate that research "impact" is not a considered guideline for publication, this statement of "strong evidence" is a stretch. I'd suggest removing the "strong" adjective here.

* Fig 5: This plot is hard to read (both in image quality and presentation). The addition of a contour plot, estimated 3-d surface, or some other positional indicator(s) (e.g., a line drawn from the horizontal plane vertically to each point on the vertical axis) would greatly increase readability.

* Section 5.2.1 / Table 6: What part of the data are these values drawn from? The 30% of D_test described in Section 4.3? Table 3 states that there are 13,765 pull requests in the test set, but there are 3,610 data points in each cell of Table 6.

Misc. Minor Comments

-------

* Line 197-200: "one annotator labelled 200 pull requests, and another annotator labelled an a different set of 100 pull requests"; it is not clear that the "another annotator" is the same annotator for which ICC was calculated in prior sentences.

Grammar / Typos:

* Line 132: "that adopt pull request model"

* Line 199: "... another annotator labelled an a different ..."

* Line 234: "For each comment evaluator makes on ..."

* Line 256: "i.e." should be italicized

* Line 450: "i.e." should be italicized

**********

6. PLOS authors have the option to publish the peer review history of their article (what does this mean?). If published, this will include your full peer review and any attached files.

If you choose “no”, your identity will remain anonymous but your review may still be made public.

Do you want your identity to be public for this peer review? For information about this choice, including consent withdrawal, please see our Privacy Policy.

Reviewer #1: No

[NOTE: If reviewer comments were submitted as an attachment file, they will be attached to this email and accessible via the submission site. Please log into your account, locate the manuscript record, and check for the action link "View Attachments". If this link does not appear, there are no attachment files to be viewed.]

While revising your submission, please upload your figure files to the Preflight Analysis and Conversion Engine (PACE) digital diagnostic tool, https://pacev2.apexcovantage.com/. PACE helps ensure that figures meet PLOS requirements. To use PACE, you must first register as a user. Registration is free. Then, login and navigate to the UPLOAD tab, where you will find detailed instructions on how to use the tool. If you encounter any issues or have any questions when using PACE, please email us at figures@plos.org. Please note that Supporting Information files do not need this step.

Revision 1

Thanks to the editor and the reviewers for very helpful comments.

We have addressed all the comments in the revision. For each comment, we identify the page number of the manuscript on which we made the pertinent changes; on the specified page of the manuscript with tracked changes, we have inserted an arrow in the margin (with the specified tag) that points approximately to the changed text.

Attachments
Attachment
Submitted filename: Response to Reviewers.pdf
Decision Letter - Tiago P. Peixoto, Editor

A Network-Centric Approach for Estimating Trust between Open Source Software Developers

PONE-D-19-20119R1

Dear Dr. Murukannaiah,

We are pleased to inform you that your manuscript has been judged scientifically suitable for publication and will be formally accepted for publication once it complies with all outstanding technical requirements.

Within one week, you will receive an e-mail containing information on the amendments required prior to publication. When all required modifications have been addressed, you will receive a formal acceptance letter and your manuscript will proceed to our production department and be scheduled for publication.

Shortly after the formal acceptance letter is sent, an invoice for payment will follow. To ensure an efficient production and billing process, please log into Editorial Manager at https://www.editorialmanager.com/pone/, click the "Update My Information" link at the top of the page, and update your user information. If you have any billing related questions, please contact our Author Billing department directly at authorbilling@plos.org.

If your institution or institutions have a press office, please notify them about your upcoming paper to enable them to help maximize its impact. If they will be preparing press materials for this manuscript, you must inform our press team as soon as possible and no later than 48 hours after receiving the formal acceptance. Your manuscript will remain under strict press embargo until 2 pm Eastern Time on the date of publication. For more information, please contact onepress@plos.org.

With kind regards,

Tiago P. Peixoto

Academic Editor

PLOS ONE

Additional Editor Comments (optional):

Reviewers' comments:

Reviewer's Responses to Questions

Comments to the Author

1. If the authors have adequately addressed your comments raised in a previous round of review and you feel that this manuscript is now acceptable for publication, you may indicate that here to bypass the “Comments to the Author” section, enter your conflict of interest statement in the “Confidential to Editor” section, and submit your "Accept" recommendation.

Reviewer #1: All comments have been addressed

**********

2. Is the manuscript technically sound, and do the data support the conclusions?

The manuscript must describe a technically sound piece of scientific research with data that supports the conclusions. Experiments must have been conducted rigorously, with appropriate controls, replication, and sample sizes. The conclusions must be drawn appropriately based on the data presented.

Reviewer #1: Yes

**********

3. Has the statistical analysis been performed appropriately and rigorously?

Reviewer #1: Yes

**********

4. Have the authors made all data underlying the findings in their manuscript fully available?

The PLOS Data policy requires authors to make all data underlying the findings described in their manuscript fully available without restriction, with rare exception (please refer to the Data Availability Statement in the manuscript PDF file). The data should be provided as part of the manuscript or its supporting information, or deposited to a public repository. For example, in addition to summary statistics, the data points behind means, medians and variance measures should be available. If there are restrictions on publicly sharing data—e.g. participant privacy or use of data from a third party—those must be specified.

Reviewer #1: Yes

**********

5. Is the manuscript presented in an intelligible fashion and written in standard English?

PLOS ONE does not copyedit accepted manuscripts, so the language in submitted articles must be clear, correct, and unambiguous. Any typographical or grammatical errors should be corrected at revision, so please note any specific errors here.

Reviewer #1: Yes

**********

6. Review Comments to the Author

Please use the space provided to explain your answers to the questions above. You may also include additional comments for the author, including concerns about dual publication, research ethics, or publication ethics. (Please upload your review as an attachment if it exceeds 20,000 characters)

Reviewer #1: Thank you to the author for addressing all listed concerns explicitly, concisely, and concretely. Particularly, it appears extensive additional effort was expended in order to completely address my concerns (e.g., additional experiments, variance-informed comparisons, additional baselines, etc.), which is greatly appreciated.

This manuscript is now complete and all my concerns have been addressed.

**********

7. PLOS authors have the option to publish the peer review history of their article (what does this mean?). If published, this will include your full peer review and any attached files.

If you choose “no”, your identity will remain anonymous but your review may still be made public.

Do you want your identity to be public for this peer review? For information about this choice, including consent withdrawal, please see our Privacy Policy.

Reviewer #1: No

Formally Accepted
Acceptance Letter - Tiago P. Peixoto, Editor

PONE-D-19-20119R1

A Network-Centric Approach for Estimating Trust between Open Source Software Developers

Dear Dr. Murukannaiah:

I am pleased to inform you that your manuscript has been deemed suitable for publication in PLOS ONE. Congratulations! Your manuscript is now with our production department.

If your institution or institutions have a press office, please notify them about your upcoming paper at this point, to enable them to help maximize its impact. If they will be preparing press materials for this manuscript, please inform our press team within the next 48 hours. Your manuscript will remain under strict press embargo until 2 pm Eastern Time on the date of publication. For more information please contact onepress@plos.org.

For any other questions or concerns, please email plosone@plos.org.

Thank you for submitting your work to PLOS ONE.

With kind regards,

PLOS ONE Editorial Office Staff

on behalf of

Dr. Tiago P. Peixoto

Academic Editor

PLOS ONE

Open letter on the publication of peer review reports

PLOS recognizes the benefits of transparency in the peer review process. Therefore, we enable the publication of all of the content of peer review and author responses alongside final, published articles. Reviewers remain anonymous, unless they choose to reveal their names.

We encourage other journals to join us in this initiative. We hope that our action inspires the community, including researchers, research funders, and research institutions, to recognize the benefits of published peer review reports for all parts of the research system.

Learn more at ASAPbio .