intel
[enh] Conda recipe update and CI OS version pinning
Description
Scikit-learn-intelex conda recipe update.
Changes:
- Update content of recipe, building and testing scripts to sync with conda-forge feedstock
- Add recipe building on Linux and Windows to CI
- Pin CI images versions
- Add hint for CMake to get python executable from $PYTHON if set
Conda-forge checks: https://github.com/conda-forge/scikit-learn-intelex-feedstock/pull/38
Checklist to comply with before moving PR from draft:
PR completeness and readability
- [x] I have reviewed my changes thoroughly before submitting this pull request.
- [x] I have commented my code, particularly in hard-to-understand areas.
- [x] I have updated the documentation to reflect the changes or created a separate PR with update and provided its number in the description, if necessary.
- [x] Git commit message contains an appropriate signed-off-by string (see CONTRIBUTING.md for details).
- [x] I have added a respective label(s) to PR if I have a permission for that.
- [x] I have resolved any merge conflicts that might occur with the base branch.
Testing
- [x] The unit tests pass successfully.
- [x] I have run it locally and tested the changes extensively.
Performance
- [x] I have measured performance for affected algorithms using scikit-learn_bench and provided at least summary table with measured data, if performance change is expected.
- [x] I have provided justification why performance has changed or why changes are not expected.
Question: are the commands in meta.yaml for executing tests actually needed if it will end up calling the run_test.sh/bat scripts in the end? E.g. do they execute somewhere else too?
@Alexsandruss This was merged with an error in the WindowsCondaEnv Python3.12_Sklearn1.5 step
@Alexsandruss This was merged with an error in the WindowsCondaEnv Python3.12_Sklearn1.5 step
The error appears to be sporadic though.