-
Notifications
You must be signed in to change notification settings - Fork 14
adding cran maintainers 2017 info from Pinto et al. 2018 #108
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for this update, Anna, it's looking good. I've made several comments, but they are minor and hopefully quick to address.
One thing I wondered about is the "Author Demographics 2010" section. I know you didn't work on this, but my guess is it's the same as the Pinto et al study in that it is actually Maintainers who were surveyed. Can you look into it and either change "author" to maintainer in that section, so that it is clear they are talking about the same thing (apart from Pinto et al's restriction to scientists) or add a clarifying comment if Mair et al did ask all authors of the package to respond.
If you don't have time to look into the Mair et al study right now, you could just fix the minor issues so we can merge this PR, then add an issue to check the Mair et al study later.
…nto pinto_et_al
Heather, I have incorporated your comments in commit 56e8927 and 910a587. I chose to use self-identified throughout. Also, I updated the plots because I realized by truncating the values, they weren't adding up to 100%. The figures in the new plots are rounded (and I also updated the text to match). I made a mistake merging the master branch into this branch, hopefully that doesn't pose a big problem. |
I took a brief look at the Mair et al. 2010 paper. The authors of the paper use the term authors to refer to the respondents of their email survey. The methodology note says "In total, we had 4,274 email addresses of R package authors. ... The platforms we used for the acquisition of the email addresses were CRAN, R-Forge, and Bioconductor. ... Note that if packages had multiple authors, emails were sent out to those who provided an email address in the package description file. In addition, in the email list we used, some package authors had multiple email addresses. Therefore, the response rate below reflects a lower bound." I am inclined to use the language that the paper authors use, but can change if maintainers is the more accurate term. |
Thanks, your reasoning makes sense to me, merged in now. |
No description provided.