OHDSI / Vocabulary-v5.0

Build process for the OHDSI Standardized Vocabularies. Currently not available as independent release.
The Unlicense
213 stars 73 forks source link

How LOINC panels affect its hierarchy: COVID LOINC Group LG51017-8 SARSCoV2 detection contains not SARS-2 tests #627

Open xj2193 opened 2 years ago

xj2193 commented 2 years ago

We noticed the new LOINC group 36662633 have 102 descendants Lab Tests. However only 80 are included in the descendants of 756055. Please see the following queries. Could you let me know if there's anything that I missed or it's an issue needs to be fixed? Thanks very much!

select distinct * from concept_ancestor ca where ca.ancestor_concept_id=756055 and descendant_concept_id in (select descendant_concept_id from concept_ancestor where ancestor_concept_id = 36662633)

TinyRickC137 commented 2 years ago

Hello, @xj2193 Concept ancestor table only has classificational and standard concepts. Probably, the missing LOINCs are deprecated non-standard ones.

Also, these types of questions better are asked on OHDSI Forum, because they are not really an issue.

Best regards!

cukarthik commented 2 years ago

Hi @TinyRickC137 ,

I believe this is an issue b/c LOINC has a group (36662633) to classify SAR-COV2. OHDSI created a grouping when one did not exist using an OMOP Extension code (756055). If I run this query, I see 24 standard LOINC codes in the Measurement domain that are not part of parent code (756055):

select distinct *
from concept_ancestor ca
where ca.ancestor_concept_id=36662633
and descendant_concept_id not in
(select descendant_concept_id 
from concept_ancestor as ca2
join concept as c on c.concept_id=ca2.descendant_concept_id
where ca2.ancestor_concept_id = 756055 and c.standard_concept='S' and c.domain_id='Measurement')

@mik-ohdsi can you confirm?

MariaRohozhkina commented 2 years ago

There is no division between SARS-COV2 and SARS-related coronavirus in LOINC hierarchy. But we separate it, because SARS-related coronavirus is group of viruses not only SARS-COV2, so part of LOINC group 36662633 has another hierarchy. Please, take descendants of 705104 and they should appear.
Also, we don't include panels, since they are not used for coding and are not consistent in composition.

mik-ohdsi commented 2 years ago

So, @cukarthik and @MariaRohozhkina. Is the conclusion that we should clean up a little and retire that OMOP Extension concept now and map it over to a better one from LOINC, updating / adding mappings to that grouping concept we chose from that fairly decent LOINC concept list for SARS CoV2 detection plus more? Or is our finding that there still is no good replacement for that OMOP Extension? And can you help me here, as I unfortunately did not fully understand how the general Coronavirus detection OMOP extension concept is helping us. I probably did not understand how they differ when compared to the LOINC hierarchy. Thanks!

Alexdavv commented 2 years ago

retire that OMOP Extension concept now and map it over to a better one from LOINC

No. The LOINC grouper is only for LOINC concepts. By the definition of the ontology we build, LOINC can't be above the SNOMED and others.

from that fairly decent LOINC concept list

It's not. The ones we excluded are not exactly the SARS-2 detection.

Or is our finding that there still is no good replacement for that OMOP Extension?

Right. Only SNOMED could have it. Or OMOP Extension.

general Coronavirus detection OMOP extension concept is helping us

If you want not just SARS-2, but also SARS-1 and others, you pick it.

how they differ when compared to the LOINC hierarchy

Look at this concept SARS-related coronavirus E gene [Presence] in Saliva (oral fluid) by NAA with probe detection. It's not exactly about the SARS-2. image

And it's not even linked to the SARS-2 directly. image

But because the panels are not consistent... image

...it rolls up to the SARS-2. image

We better need to check how LOINC panels affect its hierarchy.