Skip to content
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

docs: vignette on the rationale for the algorithm #63

Merged
merged 6 commits into from
Mar 22, 2024

Conversation

lwjohnst86
Copy link
Member

This was mostly copied over from @Aastedet thesis page, and revised to fit better as a vignette and to have a lighter/less academic narrative.

@lwjohnst86 lwjohnst86 self-assigned this Mar 15, 2024
Copy link
Collaborator

@Aastedet Aastedet left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

title = {{Algoritmer for udvalgte kroniske sygdomme og svære psykiske lidelser [English: Algorithms for selected chronic diseases and serious mental illnesses]}},
year = {2022},
url = {https://www.esundhed.dk/-/media/Files/Publikationer/Emner/Operationer-og-diagnoser/Udvalgte-kroniske-sygdomme-svaere-psykiske-lidelser/Algoritmer-for-Udvalgte-Kroniske-Sygdomme-og-svre-psykiske-lidelser-RUKS-2022.ashx},
}
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same deal here, This link is dead.

Currently, the working link is: https://www.esundhed.dk/-/media/Files/Publikationer/Emner/Operationer-og-diagnoser/Udvalgte-kroniske-sygdomme-svaere-psykiske-lidelser/Algoritmer-for-Udvalgte-Kroniske-Sygdomme-og-svre-psykiske-lidelser.ashx

Going by the name of this URL, this link might stay alive even if they update the documentation (there's no year-reference in the URL). I guess we should update the links to this one?

title = {{Algoritmer for udvalgte kroniske sygdomme og svære psykiske lidelser [English: Algorithms for selected chronic diseases and serious mental illnesses]}},
year = {2016},
url = {https://www.esundhed.dk/-/media/Files/Dokumentation/Register-for-Udvalgte-Kroniske-Sygdomme-og-Svaere-Psykiske-Lidelser/29_Algoritmer_for_RUKS.ashx},
}
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The links to previous RSCD/RUKS documentation are killed every time they update the documentation, so the links to DHDA2016 and SDHDA2022lastaccessed7Feb2023 are now dead. The original documentation is inaccessible to the public and all you have to go by is the "revision log" table in the current word document, which is not really sufficient for reproducibility.

Copy link
Contributor

@signekb signekb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Very nice! I have added a couple of suggestions :)

vignettes/rationale.Rmd Outdated Show resolved Hide resolved
vignettes/rationale.Rmd Outdated Show resolved Hide resolved
vignettes/rationale.Rmd Outdated Show resolved Hide resolved
Register, which was established in 2006 [@Carstensen2011] but then
discontinued in 2012.

The next resource is the Register of Selected Chronic Diseases (RSCD),
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Add source for RSCD?
The Danish Health Data Authority. The register of selected chronic diseases; 2014. Available from: https://www.esundhed.dk/Dokumentation/DocumentationExtended?id=29. Accessed November 25, 2022.


General-purpose registers and other administrative databases often
provide the basis of diabetes epidemiology, but they rarely contain
validated diabetes-specific data, which may introduce bias in studies
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe relevant to note what kinds of biases?

vignettes/rationale.Rmd Outdated Show resolved Hide resolved
vignettes/references.bib Outdated Show resolved Hide resolved
vignettes/references.bib Outdated Show resolved Hide resolved
Co-authored-by: Signe Kirk Brødbæk <[email protected]>
@lwjohnst86 lwjohnst86 merged commit e111460 into main Mar 22, 2024
0 of 3 checks passed
@lwjohnst86 lwjohnst86 deleted the docs/explanation-docs branch March 22, 2024 11:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

Successfully merging this pull request may close these issues.

3 participants