[BUG]: Why deepchecks is not installed with pycaret[full]

pycaret / pycaret

An open-source, low-code machine learning library in Python

https://www.pycaret.org

MIT License

8.93k stars 1.77k forks source link

[BUG]: Why deepchecks is not installed with pycaret[full] #2882

Closed moezali1 closed 2 years ago

moezali1 commented 2 years ago

pycaret version checks

[X] I have checked that this issue has not already been reported here.
[X] I have confirmed this bug exists on the latest version of pycaret.
[X] I have confirmed this bug exists on the master branch of pycaret (pip install -U git+https://github.com/pycaret/pycaret.git@master).

Issue Description

Why deepchecks is not installed with pycaret[full]? autoviz explainerdashboard all of them are installed but not deepchecks. Is there a specific reason to this?

Reproducible Example

Expected Behavior

deepchecks should be installed with pycaret[full].

Actual Results

Not getting installed.

Installed Versions

3.0

ngupta23 commented 2 years ago

Per August 22nd, @tvdboom will

[x] add it to "Analysis" section in requirements-optional.
[x] Add optional library check for deepchecks in the code where this library is referenced.

tvdboom commented 2 years ago

Done in #2866

moezali1 commented 2 years ago

Will not be done because deepchecks is not yet compatible with latest version of plotly. deepchecks is pinned to plotly==5.5.0 and some other dependencies of pycaret requires the latest version. Hence deepchecks is skipped from requirements until that is sorted.

ItayGabbay commented 2 years ago

Hey @moezali1 Deepchecks maintainer here. Just letting you guys know we've finally released a new version (0.9.2) that relaxes this strict plotly requirement. Deepchecks is now compatible with the latest version of plotly.

tvdboom commented 2 years ago

@ItayGabbay Thanks for the update. I added the dependency in #3070