Closed dvorst closed 1 year ago
Can we get a merge ASAP on this? ELI5 is now broken for most use-cases since you know, most people are running this on sklearn models...
Also breaks my HF space: https://huggingface.co/spaces/Hellisotherpeople/Interpretable_Text_Classification_And_Clustering
@Hellisotherpeople this was merged in https://github.com/eli5-org/eli5/pull/19 and released in 0.12, thanks for a reminder
When using Jinja2 3.1.2 and scikit-learn 1.0.2, the following errors occurred:
The changes in 'html.py', 'permutation_importance.py' and 'transform.py' are such that these errors are resolved while maintaining support for older versions of jinja2 and scikit-learn.
'test_import.py' is used to test importing eli5 for both newer and older dependency versions.
Also see: https://github.com/TeamHG-Memex/eli5/issues/417