dependencies for conda package testing (facebookresearch#208)

Summary: Now opencv and tensorboard are not official dependencies, make sure they are available for the conda tests. This is part of the followup to facebookresearch#198 (conda not installing nicely), specifically PRs facebookresearch#202 and facebookresearch#205 which removed them as deps. Pull Request resolved: facebookresearch#208 Reviewed By: prigoyal Differential Revision: D26736846 Pulled By: bottler fbshipit-source-id: 0a0b8af1d05067d82f2cd946faa0faeec0be094d
ArrowLuo · Mar 1, 2021 · 859d8f6 · 859d8f6
1 parent 3e85241
commit 859d8f6
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/dev/packaging/vissl_conda/build_conda.sh b/dev/packaging/vissl_conda/build_conda.sh
@@ -13,4 +13,6 @@ export BUILD_VERSION=$build_version
 
 # We allow the vissl channel to get the apex package.
 # We specify it with a full url to avoid a name clash with a local directory called vissl.
-conda build -c https://conda.anaconda.org/vissl -c iopath -c conda-forge -c pytorch -c defaults --no-anaconda-upload --python "$PYTHON_VERSION" --output-folder "$packaging/out" "$packaging/vissl"
+# Having defaults before conda-forge is so that the tensorboard used in the
+# tests will work.
+conda build -c https://conda.anaconda.org/vissl -c iopath -c pytorch -c defaults -c conda-forge --no-anaconda-upload --python "$PYTHON_VERSION" --output-folder "$packaging/out" "$packaging/vissl"
diff --git a/dev/packaging/vissl_conda/vissl/meta.yaml b/dev/packaging/vissl_conda/vissl/meta.yaml
@@ -43,6 +43,8 @@ test:
   requires:
     - ca-certificates
     - cudatoolkit=10.1
+    - opencv
+    - tensorboard
   commands:
     - python -m unittest discover -v -s tests
     - ./dev/run_quick_tests.sh