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

prior observation for matched cohort will be the same as sample for matchCohorts() #415

Open
daniellenewby opened this issue Dec 19, 2024 · 1 comment

Comments

@daniellenewby
Copy link

I have a cancer cohort with patients with a diagnosis of cancer with at least 365 days of prior history. I want to match these people using the matchCohorts(). However, will the matched cohort of "controls" without cancer also have 365 days of prior observation? From the function i do not think this is the case. If not then there could be the potential for sample cohort (people with cancer) to be matched to the matched cohort (people without cancer) who have less than 365 days of observation. Therefore when comparing characteristics between the matched and sample might introduce a little bias? It might not make too much difference as it is supposed to be random matching but i think it needs discussion on if this should be included the prior history from the sample or somehow can be detected from the sample cohort and applied to the matched cohort. Thanks!

@nmercadeb
Copy link
Collaborator

Maybe we can add a subsetCohort argument to pick controls only from that cohort, using its entry and exit dates?
This way, if some inlcusion/exclusion criteria was applied to the target cohort, the user could apply it to a population cohort that would then be used to find controls

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants