search

langcog / metalab2

MetaLab -- Community-augmented meta-analysis
http://metalab.stanford.edu/
MIT License
21 stars 8 forks source link

Have a peer-reviewed subset selector for all MAs? #55

Open christinabergmann opened 6 years ago

christinabergmann commented 6 years ago

Some people argue against including unpublished data or information from conference abstracts, should we make subsetting to peer reviewed a standard option?

mcfrank commented 6 years ago

It's as easy as a checkbox, so I don't see why not, unless we think that they shouldn't be able to do that.

On Fri, Mar 9, 2018 at 5:18 AM, Christina Bergmann <notifications@github.com

wrote:

Some people argue against including unpublished data or information from conference abstracts, should we make subsetting to peer reviewed a standard option?

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/langcog/metalab2/issues/55, or mute the thread https://github.com/notifications/unsubscribe-auth/ABGul4FoNnk2MIM2KvCC__6Cn36mNN4rks5tcoEcgaJpZM4SkQEG .

christinabergmann commented 6 years ago

OK, should we specify it in the datasets.yaml or make it standard?

juliacarbajal commented 6 years ago

I'm working on this but I have a small question regarding the implementation: In order to use subsetting, the variable must be coded as TRUE or FALSE, but at the moment the peer_reviewed column is coded as yes or no. We can either change this manually in the existing datasets (and modify the template for future datasets), or the variable transformation can be done in the code. I think I would prefer the first option (changing the datasets & template), what do you think?

christinabergmann commented 6 years ago

Sounds fine to me, afaik no metalab scripts use this currently.

On Wed, 14 Nov 2018 at 17:52, Julia notifications@github.com wrote:

I'm working on this but I have a small question regarding the implementation: In order to use subsetting, the variable must be coded as TRUE or FALSE, but at the moment the peer_reviewed column is coded as yes or no. We can either change this manually in the existing datasets (and modify the template for future datasets), or the variable transformation can be done in the code. I think I would prefer the first option (changing the datasets & template), what do you think?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/langcog/metalab2/issues/55#issuecomment-438734370, or mute the thread https://github.com/notifications/unsubscribe-auth/ALVmky052mA82NQvAZecsgjq59wcYhHYks5uvEosgaJpZM4SkQEG .

-- Researcher at the Max Planck Institute for Psycholinguistics, Language Development Department, Nijmegen, The Netherlands Website: sites.google.com/site/chbergma Blog: cogtales.wordpress.com

shotsuji commented 6 years ago

Either option sounds fine to me!

Am Mi., 14. Nov. 2018 um 17:52 Uhr schrieb Christina Bergmann < notifications@github.com>:

Sounds fine to me, afaik no metalab scripts use this currently.

On Wed, 14 Nov 2018 at 17:52, Julia notifications@github.com wrote:

I'm working on this but I have a small question regarding the implementation: In order to use subsetting, the variable must be coded as TRUE or FALSE, but at the moment the peer_reviewed column is coded as yes or no. We can either change this manually in the existing datasets (and modify the template for future datasets), or the variable transformation can be done in the code. I think I would prefer the first option (changing the datasets & template), what do you think?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/langcog/metalab2/issues/55#issuecomment-438734370, or mute the thread < https://github.com/notifications/unsubscribe-auth/ALVmky052mA82NQvAZecsgjq59wcYhHYks5uvEosgaJpZM4SkQEG

.

-- Researcher at the Max Planck Institute for Psycholinguistics, Language Development Department, Nijmegen, The Netherlands Website: sites.google.com/site/chbergma Blog: cogtales.wordpress.com

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/langcog/metalab2/issues/55#issuecomment-438734735, or mute the thread https://github.com/notifications/unsubscribe-auth/APkTmyP79MORA24TvBW7KS-XS7jt25JJks5uvEpogaJpZM4SkQEG .

-- Postdoctoral Researcher Laboratoire de Sciences Cognitives et Psycholinguistique, Paris, France Website: https://sites.google.com/site/tsujish/home Blog: https://cogtales.wordpress.com/