openkinome / kinoml

Structure-informed machine learning for kinase modeling

https://openkinome.org/kinoml/

MIT License

51 stars 21 forks source link

Refactor multiprocessing handling of Featurizers #76

Closed schallerdavid closed 3 years ago

schallerdavid commented 3 years ago

Description

Currently Featurizers support multiprocessing in the .featurize() method. This PR aims to move to the initialization of Featurizers. This will also make things easier for Pipeline etc. Featurizers, which aggregate information from different Featurizers.

Todos

[x] make sure dask client can be passed via init
[x] move all other multiprocessing stuff to init
[x] adapt all Featurizers to the new behavior

Status

[ ] Ready to go

codecov-commenter commented 3 years ago

Codecov Report

Merging #76 (f5c570b) into master (6c0dc64) will increase coverage by 0.01%. The diff coverage is 85.07%.