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

We need to update documentation and comments here to reflect that parent_codes must be formatted in a specific way. #36

Closed
mmcdermott opened this issue Aug 1, 2024 · 4 comments · Fixed by #47

Comments

@mmcdermott
Copy link
Contributor

("parent_codes", pa.list_(pa.string())),

@EthanSteinberg is the following language accurate for their constraints?

parent_codes must be a list of strings, each string being a parent code in the OMOP concept space corresponding to this code. Here, a "parent" means that this code must be a realization of either the referenced OMOP concept or a more specific example of said concept. OMOP concepts are referenced in these strings via the format "$VOCABULARY_NAME/$CONCEPT_NAME". For example: "ICD9CM/487.0"

@EthanSteinberg
Copy link
Collaborator

I don't think that's quite right, as parent_codes should, and will, contain non-OMOP concepts to handle dataset specific ontology structures.

parent_codes must be a list of strings, each string being a higher level code that represents a generalization of the provided code. Parent codes can use any structure, but is recommended that they reference OMOP concepts whenever possible, to enable use of more generic labeling functions and OHDSI tools. OMOP concepts are referenced in these strings via the format "$VOCABULARY_NAME/$CONCEPT_NAME". For example: "ICD9CM/487.0"

@mmcdermott
Copy link
Contributor Author

Can you suggest some updated language @EthanSteinberg? I want to make sure it is accurately described.

@EthanSteinberg
Copy link
Collaborator

What's wrong with the language I posted above in my original reply?

@mmcdermott
Copy link
Contributor Author

I apologize -- I didn't realize you had modified the quoted text, I thought you were just commenting on that block specifically in your reply. That text looks great. Thank you!

@mmcdermott mmcdermott linked a pull request Aug 16, 2024 that will close this issue
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 a pull request may close this issue.

2 participants