TST: loosen requirements for scikit-learn tests

interpretml / interpret

Fit interpretable models. Explain blackbox machine learning.

https://interpret.ml/docs

MIT License

6.04k stars 714 forks source link

TST: loosen requirements for scikit-learn tests #520

Closed DerWeh closed 2 months ago

DerWeh commented 2 months ago

This is a fixup for #518 We only need an up-to-date version for running tests, not for usage of the library.

Furthermore, it is a bit more robust to skip tests based on names instead of comparing functions. This avoids errors, if test aren't available yet in older versions of scikit-learn.

Sorry for this fixup, these issues slipped my mind when I create the original PR...

paulbkoch commented 2 months ago

Thanks @DerWeh, this looks great.