search

RADar-AZDelta / Rabbit-in-a-Blender

An ETL pipeline to transform your EMP data to OMOP.
https://radar-azdelta.github.io/Rabbit-in-a-Blender/
GNU General Public License v3.0
11 stars 3 forks source link

Suggestion for small change in table order of ETL #48

Closed lbertelo01 closed 4 months ago

lbertelo01 commented 4 months ago

image

The vocabulary table should be one of the first tables to run, because if a table uses custom concepts, it checks if the vocabulary used (in our case AZDELTA) is present in the vocabulary table. With the order used now, we get a problem with the custom concepts in the fact_relationship table.

pjlammertyn commented 4 months ago

solved in commit https://github.com/RADar-AZDelta/Rabbit-in-a-Blender/commit/86147b3582a55d19e57514da75d713d1c653af53