search

interpretml / interpret-text

A library that incorporates state-of-the-art explainers for text-based machine learning models and visualizes the result with a built-in dashboard.
MIT License
413 stars 68 forks source link

fix nightly build failures #184

Closed imatiach-msft closed 2 years ago

imatiach-msft commented 2 years ago

fixes the nightly build failures which are blocking a new release to pypi:

AttributeError: 'PapermillExecutePreprocessor' object has no attribute 'setup_preprocessor'

imatiach-msft commented 2 years ago

link to new nightly build triggered for this PR:

https://dev.azure.com/responsibleai/interpret-text/_build/results?buildId=13394&view=results

imatiach-msft commented 2 years ago

started another nightly build with version bump to scikit-learn: https://dev.azure.com/responsibleai/interpret-text/_build/results?buildId=13399&view=results I hope this might speed up the tests as it looks like the nightly build spends most of the time right now in the fit method of the classical text explainer

imatiach-msft commented 2 years ago

note this nightly build with increase in test time seems to be green, so the tests are just taking much more time right now: https://dev.azure.com/responsibleai/interpret-text/_build/results?buildId=13396&view=results

imatiach-msft commented 2 years ago

Updated PR to add a tolerance parameter and reduce the tolerance in the sample notebook to reduce execution time. Note the reduced tolerance in the notebook does not seem to have an effect on the metrics since the notebook tests are still passing.