search

ARCH-commons / i2p-transform

i2b2 to PCORnet Common Data Model Transformation - requires i2b2 PCORnet ontology
Other
3 stars 16 forks source link

Griffin's Smoking status character limit issue #48

Closed matthewjoss closed 7 years ago

matthewjoss commented 7 years ago

Griffin: BIDMC uses the smoking status "\PCORI\VITAL\TOBACCO\SMOKING\YES\", which has the PCORI_BASECODE = "YES". This code is three characters, but pmnVITAL.SMOKING field is only two characters. As a result, I get an error when it tries to insert "YES" as the smoking status. For now, I've converted "YES" to "01", but that isn't exactly correct. How should I be handling this?

matthewjoss commented 7 years ago

Jeff said: For #2... @GriffinWeber should be using a more specific code than YES, either 01 ('Current every day') or 05 ('Smoker, status unknown'). We need to update our mapping docs. @matthewjoss - can you do this?

matthewjoss commented 7 years ago

I did this a couple of weeks ago