opentargets / issues

Issue tracker for Open Targets Platform and Open Targets Genetics Portal
https://platform.opentargets.org https://genetics.opentargets.org
Apache License 2.0
12 stars 2 forks source link

Improving ETL architecture for Pharmacogenomics around Drug Utils #3326

Open mbdebian opened 1 month ago

mbdebian commented 1 month ago

While working in the pharmacogenomics ETL step, I've found the helper function MapDrugId which only client is the pharmacogenomics step, at this line.

It seems to be a helper particular to the pharmacogenomics step, taking into account that the ETL step lives in its own folder, helpers that are meant to be specific for that step should actually reside within its packaging scope.

After further checks, the only DrugUtils client is pharmacogenomics.

Please, refactor the code accordingly.