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

New function to subset results #937

Open
wants to merge 3 commits into
base: develop
Choose a base branch
from

Conversation

gowthamrao
Copy link
Member

This is a function i have found useful.

Use case: During phenotype development and evaluation we develop and evaluate many candidate cohort definitions. But after the process is complete - we reject many cohorts and just want to remove them. We do not intend them to be shared for various reasons.

This function, takes a previously computed cohort diagnostics output and subsets it to a set of cohortIds given by the user, creates a new zip file that can then be shared.

If this is a useful function, but requires some changes - please feel free to make changes .

@gowthamrao gowthamrao marked this pull request as draft September 20, 2022 21:32
@gowthamrao gowthamrao marked this pull request as ready for review September 20, 2022 21:36
),
col_types = readr::cols())

if ("cohort_id" %in% colnames(dataFromZip)) {
Copy link
Collaborator

Choose a reason for hiding this comment

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

I'm not sure this works for all files - what about cohort relationships?

Copy link
Member Author

Choose a reason for hiding this comment

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

Should be possible to support it. But you think this function is useful and want to add to the package?

Copy link
Member Author

Choose a reason for hiding this comment

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

cohort Relationship has cohort_id - so it should work like any other table correct?

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

Successfully merging this pull request may close these issues.

None yet

2 participants