-
-
Notifications
You must be signed in to change notification settings - Fork 25
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
Replace R code by Notebook #13
Conversation
Thanks @terrymyc! Thanks for your contribution. A couple of additions and remarks are listed below:
|
There is a discrepancy in the process between this dataset and Wilson dataset. For this dataset, 8 papers are directly included after abstract screening without full-text screening, whereas the Wilson dataset does not. So, the inclusion indicators are different in two datasets. Should we have a unified criteria for this? |
datasets/Van_de_Schoot_PTSD/convert_clean_schoot_lgmm_ptsd.ipynb
Outdated
Show resolved
Hide resolved
Good point. For now, I don't have an idea to unify this. I think we should consider these as "included after full-text reading", as we do in other datasets. |
@Rensvandeschoot decided to continue with the 38 inclusions. This means that the citation is incorrect. @GerbrichFerdinands can you drop the correct link to the paper here? |
This also means that the RIS files are stored in the wrong repository on OSF. @terrymyc, please wait till the files are moved to the correct repository. @Rensvandeschoot |
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.
Nice work Terry! ✨
Your code is very readable. I attached a few suggestions for improvement!
datasets/Van_de_Schoot_PTSD/convert_clean_schoot_lgmm_ptsd.ipynb
Outdated
Show resolved
Hide resolved
"# 62 with both missing titles and abstracts\n", | ||
"print(\"Number of papers with missing title AND abstract: \", (all_title.isna() & all_data['abstract'].isna()).sum(), \"\\n\")\n", | ||
"\n", | ||
"# 764 with either missing titles or abstracts\n", |
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.
I get 766!
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.
I changed the statistics to summarize the missingness based on a clean dataset instead of dataset with duplicates. So, it becomes even less than 766.
datasets/Van_de_Schoot_PTSD/convert_clean_schoot_lgmm_ptsd.ipynb
Outdated
Show resolved
Hide resolved
datasets/Van_de_Schoot_PTSD/convert_clean_schoot_lgmm_ptsd.ipynb
Outdated
Show resolved
Hide resolved
datasets/Van_de_Schoot_PTSD/convert_clean_schoot_lgmm_ptsd.ipynb
Outdated
Show resolved
Hide resolved
datasets/Van_de_Schoot_PTSD/convert_clean_schoot_lgmm_ptsd.ipynb
Outdated
Show resolved
Hide resolved
datasets/Van_de_Schoot_PTSD/convert_clean_schoot_lgmm_ptsd.ipynb
Outdated
Show resolved
Hide resolved
Rens van de Schoot, Marit Sijbrandij, Sonja D. Winter, Sarah Depaoli & Jeroen K. Vermunt (2017) The GRoLTS-Checklist: Guidelines for Reporting on Latent Trajectory Studies, Structural Equation Modeling: A Multidisciplinary Journal, 24:3, 451-467, DOI: 10.1080/10705511.2016.1247646 |
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.
@terrymyc Can you update the citation?
No description provided.