judithabk6 / med_bench

BSD 3-Clause "New" or "Revised" License
8 stars 3 forks source link

function naming and reorganization #51

Closed judithabk6 closed 6 months ago

judithabk6 commented 7 months ago

the file benchmark_mediation should be renamed mediation

Should we include mediation in all function names or just assume it's ok as they are in the mediation module?

_convert_array_to_R and get_interactions : should be moved to a utils module?

bthirion commented 7 months ago

for plain_IPW, AIPW, (+bart) maybe add a treatment_effect module ?

houssamzenati commented 7 months ago

this has tentatively been addressed in PR #58