Add explode dataset task#121
Conversation
|
Thanks @RobinM-code, I see the advantage here. There are few issues that would prevent us from using this.
I'll wait for @javfg to comment on this. As it stands, we could merge it as a feature for the community, but as I said, I don't think we'd be able to use it in OpenTargets just now. |
|
Thanks for the detailed feedback @jdhayhurst, those are some very valid points! I agree this doesn’t fully cover Open Targets’ needs right now (especially around product separation and dataset dependencies), and the current setup still makes sense for those use cases. That said, I think this is still a nice optional addition for simpler setups or external users, so I’d be happy to see it merged as a community feature. Also worth noting this doesn’t change or break the existing approach, so there’s no need for the OT team to switch unless you want to. |
This PR adds a
explode_datasetstask.This task can be configured like this in
config.yaml:It will load all datasets defined in
datasets.yamlwithout the need to re-specify all of them inconfig.yaml