You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Just reading some code and I'm pretty sure that level2_concept_name is duplicated in src/main/resources/resources/cdmresults/sql/report/condition/treemap.sql. The second one should be level1_concept_name like in src/main/resources/resources/cdmresults/sql/report/conditionera/treemap.sql
CONCAT(
isNull(concept_hierarchy.level4_concept_name, 'NA'), '||',
isNull(concept_hierarchy.level3_concept_name, 'NA'), '||',
isNull(concept_hierarchy.level2_concept_name, 'NA'), '||',
isNull(concept_hierarchy.level2_concept_name, 'NA'), '||', <---- should be level1_concept_name
isNull(concept_hierarchy.concept_name, 'NA')
) AS concept_path,
I agree, that seems unlikely to be by design, but let me review the concept_hierarchy to see if there's something special about condition hierarchy that we'd want to not use level 1 (and if that was the case, I don't know why they just didn't omit the lvl 1 from the concatination (unless the front end expects 4 levels always). In any case, this is a good catch and let me investigate.
Activity
chrisknoll commentedon Oct 21, 2025
I agree, that seems unlikely to be by design, but let me review the concept_hierarchy to see if there's something special about condition hierarchy that we'd want to not use level 1 (and if that was the case, I don't know why they just didn't omit the lvl 1 from the concatination (unless the front end expects 4 levels always). In any case, this is a good catch and let me investigate.