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
Synthesis Version Updates #103
Comments
Resolving errors in existing hook scripts:
The following cores have a depth interval where the min and max are likely reversed:
The following studies have intervals with NA depths: "Nahlik_and_Fennessy_2016" "Langston_et_al_2022" "De_Iongh_et_al_1995" "Agawin_et_al_1996" "Townsend_and_Fonseca_1998" "Holmer_et_al_2007" "Van_Engeland_2010" "Boyd_et_al_2017" Studies with NA Coords:
Studies with NA Habitat:
Studies with Modeled C:
Synthesis and Post-Processing
|
We need a QAQC function to catch studies that have no associated citation Some starter code: no_citations <- ccrcn_synthesis$cores %>% if (nrow(no_citations) > 0) { unique(no_citations$study_id) |
Here are the QAQC results for our current version of the synthesis. It looks like a lot, but I think it's fairly small stuff that we can knock out! For example, theres a bunch of columns starting "..." which resulted from tables being output using write.csv() without specifying that row.names = F (idk why the default is set to true, its annoying) to prevent it from creating a column with the row number index included. If you spot stuff that is related to datasets you've worked on you can go for those quick fixes, or we can chat about some of the more nuanced things in our meeting (or whenever). Also, we are well past 10k cores, holy moly! 🎉
|
Synthesis QAQC Checks: Note: the following have now been added to the synthesis report output Depthseries
Cores
Bib
|
Thanks for renaming the citation tables! Some are still getting flagged in the synthesis QA and it looks like it's because the study_id was left out. (Though the citation table for Drake 2024 may still be missing from the derivative folder). So one more annoying edit there for the following: [1] "Stahl_et_al_2024" "Palinkas_and_Engelhardt_2024" |
@BettsH Could you take a look at the coordinates for Bukoski et al 2017 in the Sanderman synthesis? They were made fuzzy per the authors request, but there are a few that have ended up in Laos when they should be in Vietnam (so, a bit too fuzzy). Maybe check out the original paper or the supplementary data and/or use google earth engine to see if we can't update those so they a least get assigned the right country. Cores in question: "M1566" "M1567" "M1568" "M1569" "M1570" "M1571" "M1572" "M1573" "M1574" "M1575" "M1576" "M1577" "M1578" |
@cheneyr two tasks for you (if you haven't already done them)!
Thank you! |
This update is scheduled for March 2024
Please continue working in the develop branch!
Overall Goals:
Additional goals:
The text was updated successfully, but these errors were encountered: