NEW: Add recipe for libnvimgcodec #27546
Conversation
|
Hi! This is the friendly automated conda-forge-linting service. I wanted to let you know that I linted all conda-recipes in your PR ( Here's what I've got... For recipes/nvimgcodec/meta.yaml:
For recipes/libnvimgcodec/meta.yaml:
For recipes/libnvimgcodec/meta.yaml:
|
carterbox
force-pushed
the
nvimagecodec
branch
from
ec1eb15 to
fefb9c1
Compare
|
Hi! This is the staged-recipes linter and your PR looks excellent! 🚀 |
|
Hi! This is the friendly automated conda-forge-linting service. I just wanted to let you know that I linted all conda-recipes in your PR ( |
| - python-clang | ||
| host: | ||
| - cuda-version {{ cuda_compiler_version }} | ||
| - dlpack |
There was a problem hiding this comment.
Try adding cuda-cudart-dev to help Windows find cuda.h
| mkdir build | ||
| cd build | ||
|
|
||
| nvimg_build_args=( |
There was a problem hiding this comment.
Are there any build options missing?
| - libboost-headers | ||
| - libjpeg-turbo | ||
| {% if cuda_major == 12 %} | ||
| - libnvjpeg-dev |
There was a problem hiding this comment.
Should dependencies linked with dlopen be made optional or converted to standard dynamic links?
| - Library/lib/cmake/nvimgcodec # [win] | ||
| - etc/** # [linux] | ||
| - Library/etc/** # [win] | ||
| - extensions/*.so # [linux] |
There was a problem hiding this comment.
is PREFIX/extensions the "correct" place to install these modules?
| {% if cuda_major == 12 %} | ||
| - libnvjpeg-dev | ||
| {% endif %} | ||
| - libopencv |
There was a problem hiding this comment.
Strongly consider making opencv optional for users because it is heavy
closes #28218
Checklist
url) rather than a repo (e.g.git_url) is used in your recipe (see here for more details).