You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
elevated risk of building DSOs (.dll / .dylib / .so) that depend on conda-specific details by using conda-forge's compilation toolchain and shared libraries
It'd be convenient to get the CI image building finally moved into this repo (#7011, work in progress in #7109) before making other changes.
Then, remove conda from relevant places in .ci/ scripts, and do whatever's needed to get it passing again installing libraries from some mix of system package managers and pip.
Proposal
We should stop using
conda+conda-forgefor Python-package CI and instead do something like:pixipixipipMotivation
This project uses
conda+ theconda-forgechannel to set up environments for building and testing Python wheels.LightGBM/.ci/test.sh
Lines 114 to 120 in cf16013
That has been helpful in many ways here, especially:
But that setup has some significant drawbacks:
.dll/.dylib/.so) that depend on conda-specific details by using conda-forge's compilation toolchain and shared librariespip, and system-installed librarieslightgbmis used... by installing wheels into an environment managed withpip/uvlightgbmand its dependencies expect to be installedIn my opinion, those drawbacks significantly outweigh the benefits.
wheels, not conda packages, are built from this repo... they should be built and tested in environments built by installing wheels
Benefits of this work
See "Motivation" above, but in short:
Approach
It'd be convenient to get the CI image building finally moved into this repo (#7011, work in progress in #7109) before making other changes.
Then, remove
condafrom relevant places in.ci/scripts, and do whatever's needed to get it passing again installing libraries from some mix of system package managers andpip.Notes
Based on this conversation @borchero and I had recently: #7328 (comment)