scikit-learn-intelex icon indicating copy to clipboard operation
scikit-learn-intelex copied to clipboard

[enhancement] Add guard for sklearn fallback in sklearnex testing

Open icfaust opened this issue 1 year ago • 9 comments

Description

Sklearnex should not fall back to sklearn unless explicitly allowed to do so in sklearnex testing. This may otherwise mask problems with sklearnex code. Adding a conftest.py with proper pytest hook wrappers will enforce that sklearnex tests work for testing sklearnex functionality (and not sklearn). This also adds a pytest mark ("allow_sklearn_fallback") which will allow for this secondary verification to be turned off if necessary (and thereby is made apparent at the definition of the test).

Changes proposed in this pull request:

  • Add conftest.py and a hook into the sklearnex logger

icfaust avatar Jan 30 '24 20:01 icfaust

/intelci: run

icfaust avatar Jan 30 '24 20:01 icfaust

/intelci: run

icfaust avatar Jan 31 '24 15:01 icfaust

Guard activated against a sklearnex random forest test with GPU, will need to investigate and correct this before merge.

icfaust avatar Feb 01 '24 07:02 icfaust

/intelci: run

icfaust avatar Feb 02 '24 13:02 icfaust

/intelci: run

icfaust avatar Feb 05 '24 09:02 icfaust

Activating random forest to GPU now causes new sklearn test failures on gpu, this aspect has been split off into #1696 . The status of the gpu test failures in sklearn testing will now be re-evaluated

icfaust avatar Feb 05 '24 11:02 icfaust

Activating random forest to GPU now causes new sklearn test failures on gpu, this aspect has been split off into #1696 . The status of the gpu test failures in sklearn testing will now be re-evaluated

Please, deselect this tests here, if no fix in this PR.

samir-nasibli avatar Feb 05 '24 16:02 samir-nasibli

/intelci: run

icfaust avatar Feb 12 '24 20:02 icfaust

/intelci: run

icfaust avatar Feb 13 '24 07:02 icfaust

/intelci: run

icfaust avatar Feb 23 '24 16:02 icfaust

direct rerun was successful (green) http://intel-ci.intel.com/eed561eb-399f-f108-89a9-a4bf010d0e2e

icfaust avatar Feb 27 '24 20:02 icfaust