Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## develop #161 +/- ##
========================================
Coverage 93.47% 93.47%
========================================
Files 15 15
Lines 1165 1165
========================================
Hits 1089 1089
Misses 76 76 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
Maybe easier to maintain if a reference table is created (like a csv with fields comorbidityId, conceptId). Store the csv file in the package path, read to memory as data frame, bulk load to db, perform inner join? Reason - may make the maintaining easier. The advantage of the old approach of using ancestor_concept_id + descendants was it was more resistant to vocabulary changes. This change, that involves specifying each of the descendantConceptIds is more susceptible to vocabulary changes. |
| WHERE descendant_concept_id IN (4160276, 443392, 4131422, 4131428, 4131304, 4130839, | ||
| 4131938, 4133007, 4130842, 443252, 439392, 193144, 4312802, | ||
| 78097, 4246450, 4246451, 378087, 4312290, 140960, 4247962, | ||
| 192568, 4312023, 200959, 439751, 196053, 200348, 46273652, | ||
| 4281027, 198700, 44806773, 254591, 318096, 442182, 320342, | ||
| 434298, 434875, 373425, 199752, 72266, 4147162, 253717, | ||
| 4315806, 196925, 46270513, 4281030, 136354, 198371, 4314071, | ||
| 4158910, 78987, 432851) |
There was a problem hiding this comment.
@sfortin1 this list includes 439392, Primary malignant neoplasm. is that supposed to be an excluded concept?
| WHERE descendant_concept_id IN (201343, 196463, 193256, 4340385, 4340383, 201612, 46269816, | ||
| 200762, 192675, 4058696, 4026125, 200763, 198964, 4238978, | ||
| 201613, 199867, 4012113, 192240, 439674, 763021, 194692, | ||
| 4059284, 4064161, 194984, 46273476, 46269835, 439675, 4308946, | ||
| 196029, 4267417, 4340948, 194417, 4337543, 4340394, 4159144, | ||
| 194990, 4240725, 4135822, 4046123, 4059290, 4059299, 4059298, | ||
| 4058695, 42537742, 198683, 193693) | ||
| ; |
There was a problem hiding this comment.
these are the wrong concepts - should reflect corrected list: https://bmcmedinformdecismak.biomedcentral.com/articles/10.1186/s12911-023-02205-4#additional-information
4 participants
New CCI coding algorithm per #160.