-
-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
Add mmcv (v2 version, rename from mmcv-recipe) #22536
Conversation
OpenMMLab Computer Vision Foundation. Repo at https://github.com/open-mmlab/mmcv
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/mmcv:
For recipes/mmcv:
Documentation on acceptable licenses can be found here. |
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/mmcv:
|
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 ( |
license: Apache-2.0 AND LicenseRef-NVIDIA-License | ||
license_file: | ||
- LICENSE | ||
- LICENSES.md |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The 'NVIDIA Licenses' in LICENSES.md seem to be a mix of Nvidia Source Code License-NC
and a generic Copyright (c) 2021, NVIDIA CORPORATION & AFFILIATES. All rights reserved.
license. Is it ok to just use LicenseRef-NVIDIA-LIcense
here?
recipe-maintainers: | ||
- weiji14 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@apatsekin, would you like to be a conda-forge recipe maintainer for mmcv
(need a yes/no answer) since you were one for https://github.com/conda-forge/mmcv-recipe-feedstock? This is the mmcv>=2.0
version (because mmcv-full
has been renamed to mmcv
in v2.0).
recipe-maintainers: | |
- weiji14 | |
recipe-maintainers: | |
- apatsekin | |
- weiji14 |
@conda-forge/help-python, ready for review! Would appreciate some help with the NVIDIA license at #22536 (comment) and doublechecking the cuda/pytorch pin_compatible stuff. |
These outputs should be renamed on the existing feedstock rather than creating a new one. |
Rename package output from mmcv-full to mmcv, and build from the PyPI tar.gz file instead of the GitHub source tar.gz. Cherry-picked from conda-forge/staged-recipes#22536. Also added LICENSE and LICENSE.md files.
* updated v2.0.0 * Add mmengine to runtime deps Xref https://github.com/open-mmlab/mmcv/blob/v2.0.0/requirements/runtime.txt * Rename from mmcv-full to mmcv and build using pip install . -vv Rename package output from mmcv-full to mmcv, and build from the PyPI tar.gz file instead of the GitHub source tar.gz. Cherry-picked from conda-forge/staged-recipes#22536. Also added LICENSE and LICENSE.md files. * Use full url to mmcv package on PyPI Without the {{ name }} variable. * Partial revert to old build script Going from PyPI back to the GitHub source. * MNT: Re-rendered with conda-build 3.24.0, conda-smithy 3.23.1, and conda-forge-pinning 2023.05.16.03.36.14 * Add .dev_scripts/test_installation.py to source_files Also fix some whitespaces to reduce git diff * Trim down cuda_compiler_version if-then to just 11.2 Xref conda-forge/conda-forge.github.io#1901. * Add onnx, torchvision, pyturbojpeg to test.requires * Add tifffile * Add numpy to build requirements See if it fixes `ImportError: cannot import name '_fastCopyAndTranspose' from 'numpy.core._multiarray_umath' (/Users/runner/miniforge3/conda-bld/mmcv-recipe_1684234509266/_build_env/venv/lib/python3.10/site-packages/numpy/core/_multiarray_umath.cpython-310-darwin.so)` * Add back pytorch(-cpu) to build dependencies --------- Co-authored-by: Wei Ji <23487320+weiji14@users.noreply.github.com> Co-authored-by: conda-forge-webservices[bot] <91080706+conda-forge-webservices[bot]@users.noreply.github.com>
Ok, package renamed at conda-forge/mmcv-recipe-feedstock#19. New |
OpenMMLab Computer Vision Foundation. Repo at https://github.com/open-mmlab/mmcv
Note that the
mmcv-full
package has been renamed tommcv
in version 2.0.0 (see conda-forge/mmcv-recipe-feedstock#19 (comment) and open-mmlab/mmcv#2235). This newmmcv
v2 recipe will replace the current v1 recipe at https://github.com/conda-forge/mmcv-recipe-feedstock.Xref renaming discussion at conda-forge/conda-forge.github.io#1768 (comment) on how the old package (i.e.
mmcv-recipe
) will need to be archived once the new package (i.e.mmcv
) is done.Supersedes #19400
Checklist
url
) rather than a repo (e.g.git_url
) is used in your recipe (see here for more details).