-
Notifications
You must be signed in to change notification settings - Fork 452
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
Page on custom concept convention (Themis issue 73) #605
Conversation
Here is my humble ETLer perspective:
I am for: Setting vocabulary_concept_id to 0 and the convenion to put the abbrieviated name of institution as a part of the vocabulary name. (If given custom vocab is rebuilt we can also add version as second part of the name.) I am against: Adding requirement for the institution number as a part of vocabulary_concept_id.
Reusing concept class from other vocabs seems OK to me too - we can list the classes and have some 'Undefined' as allowed if the class is difficult to find. I would suggest to add the list of commonly used concept classes as the reference for ETL er in the documentation. |
rmd/customConcept.Rmd
Outdated
The Themis Working Group convened on October 6th and December 7th 2023 to discuss the creation this convention for creating custom concepts. | ||
|
||
## Introduction | ||
While the OMOP vocabularies are very comprehensive, it is not always possible to use concepts existing in the OMOP vocabularieas. For example, when using a vocabulary that is only used in your institution or having custom defined variables. In these cases, custom concepts can be used. Custom concepts are concepts that are not part of the OMOP vocabularies, and are only used in your institution. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
So we are defining custom concepts as never being part of OMOP vocabularies? This assumes that the missing concept is not going to be re-used and only has value to the local institution. The Jackalope concept creates custom concepts that are precoordinated standard concepts. These clearly could be re-used. Do we want to distinguish up front that our definition of "custom" excludes these other use cases for creating concepts?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
To me, custom concepts and standard concepts are mutually exclusive. Adding concepts (as standard OR non-standard) to the OHDSI vocabularies is a separate process.
There are several subtleties of course, which we might want to address or not:
- A method to go from custom concepts to a contribution to the OHDSI vocabularies. Only at this point a concept_id <2B would be assigned and possibly made standard. Always in collaboration with the Vocabulary WG.
- Once your source vocabulary has been added to the OHDSI vocabularies, then of course your mapping strategy will change and you might not have a need for custom concepts anymore.
- Custom concepts will always be more flexible than adding your concepts to the OHDSI vocabularies, as you don't have to go through the vocabularies release process.
@AgnesWW thanks for your feedback. Agree with all, except for:
Adding the institution name is fine, but I don't think we should make this a convention. This should be a decision by the local ETL team. A custom vocabulary might not be just be defined within an institution. |
Sure we can leave the naming to the vocabulary owner, they may have some internal convention already from previous work and want to keep it for consistency. |
@MaximMoinat I added this page in a separate PR: #651 because I was having trouble rebasing your fork. |
Resolving OHDSI/Themis#73
Open questions:
concept_class
from other vocabularies? e.g. 'Clinical Finding' for Conditions, 'Procedure' for Procedures? Or request an 'empty' class? --> we can use the concept class 'Undefined' (which is the concept class of concept_id 0)vocabulary_concept_id
to 0 --> provisional decision 2023-12-07: this is allowedconcept_hierarchy
. Is this indeed only for standard concepts? If allowed for custom concepts, can custom concepts be descendants/parents of standard concepts?