fklearn can be installed with multiple optional dependencies such as xgboost, catboost or lightgbm. The user can selected the needed deps so they don't need to install all of them to train a specific model. For this reason we can not have an import of one of these optional modules in a common path, in the top of the file, as it won't be optional any longer.
Description of the changes proposed in the pull request
Status
READY
Todo list
Background context
fklearn can be installed with multiple optional dependencies such as
xgboost,catboostorlightgbm. The user can selected the needed deps so they don't need to install all of them to train a specific model. For this reason we can not have an import of one of these optional modules in a common path, in the top of the file, as it won't be optional any longer.Description of the changes proposed in the pull request
This PR removes an incorrect import that was added to the common path in this PR: https://github.com/nubank/fklearn/pull/211