-
-
Notifications
You must be signed in to change notification settings - Fork 7
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Noisy warning on environment activation when ffmpeg or libopencv are installed in an environment on Linux #29
Comments
I'd be supportive of removing the warning (in https://github.com/conda-forge/ocl-icd-feedstock/blob/main/recipe/activate.sh). |
Worse still: the double square brackets [[ ]] syntax is specific to bash and running on |
fyi @xela-95 |
I'd second removing the warning, or at least logging to stderr, as common practice in UNIX. ocl-icd-feedstock/recipe/activate.sh Line 3 in dfffb6a
The fact that the logging happens on stdout will break all CLIs that activate an environment and run things programmatically, for example a e.g.: bash -lc 'conda activate /path/to/my/env && python3 -m pip list --format json' | jq gets forced to become: bash -lc 'conda activate /path/to/my/env 1>/dev/null && python3 -m pip list --format json' | jq |
fyi @flferretti |
🎆 sorry for the accidental click |
Thanks a lot @matthiasdiener ! |
Comment:
Since conda-forge/openvino-feedstock#73 has been merged, all the new environments in which I install
opencv
orffmpeg
get a big and scary warning:This warning actually comes from the
ocl-icd
package, that comes from thelibopencv --> libopenvino-intel-gpu-plugin --> ocl-icd
dependency chain. The warning is printed on every environment activation, and kind of confuses users, especially the one that do not care or even do not know anything about OpenCL. I do not have any specific opinion on OpenCL so removingocl-icd-system
may be perfectly fine for me, but do you think we could avoid this extremely noisy warning?fyi @conda-forge/ffmpeg @conda-forge/opencv @conda-forge/openvino @conda-forge/ocl-icd
The text was updated successfully, but these errors were encountered: