clinicalml / large-scale-temporal-shift-study

Code for Large-Scale Study of Temporal Shift in Health Insurance Claims. Christina X Ji, Ahmed M Alaa, David Sontag. CHIL, 2023. https://arxiv.org/abs/2305.05087
4 stars 0 forks source link

cdm_measurement_aux schema is hardcoded in sql files #4

Open egillax opened 11 months ago

egillax commented 11 months ago

I think in many cases users only have one writable schema tied to their users. To get this to work in my case I removed cdm_measurement_aux schema and replaced with {scratch_schema}. I then added the scratch_schema to the config file:

scratch_schema = `my_user_name`

Then when formatting the sql file I add this variable. I think this is more in line with situations where users can't create their own schemas but have a fixed one they can write to.

cxji commented 11 months ago

The commit above places all measurement-related tables that are created within the schema specified by measurement_aux_schema in config.py. I think it should work the same way as the scratch_schema you specified.