Skip to content

Commit 312acd8

Browse files
ProGamerGovfacebook-github-bot
authored andcommitted
Fix failing conda test, & fix a GPU test bug (#1009)
Summary: I think that the amount of Conda install timeouts can be lessened by removing unnecessary installs: * Sphinx is not used for the Conda tests, so it doesn't make sense to install it. * NumPy is a dependency of PyTorch, so there is no need to install it after installing PyTorch. The `nodejs` install seems to be what takes up the most time, and I think its where the solver sometimes fails. Using the libmamba solver seems to prevent it from failing, but it still takes vast majority of the install time (around 20 minutes) to install `nodejs`. https://www.anaconda.com/blog/a-faster-conda-for-a-growing-community I also noticed a recurring error in the GPU tests, and implemented a fix for it: ``` Errors were encountered while processing: sane-utils W: --force-yes is deprecated, use one of the options starting with --allow instead. E: Sub-process /usr/bin/dpkg returned an error code (1) ``` This PR along with #1007 should fix the test failures that Captum is experiencing. vivekmig Pull Request resolved: #1009 Reviewed By: NarineK Differential Revision: D38786344 Pulled By: vivekmig fbshipit-source-id: 508aba387f62302053945b9eb1c94d51a7c27915
1 parent 37516c1 commit 312acd8

File tree

2 files changed

+11
-3
lines changed

2 files changed

+11
-3
lines changed

.circleci/config.yml

+1
Original file line numberDiff line numberDiff line change
@@ -121,6 +121,7 @@ commands:
121121
sudo dpkg -i cuda-repo-ubuntu2004-11-4-local_11.4.2-470.57.02-1_amd64.deb
122122
sudo apt-key add /var/cuda-repo-ubuntu2004-11-4-local/7fa2af80.pub
123123
sudo apt-get update
124+
sudo dpkg --configure -a
124125
sudo apt-get --yes --force-yes install cuda
125126
126127
jobs:

scripts/install_via_conda.sh

+10-3
Original file line numberDiff line numberDiff line change
@@ -21,6 +21,10 @@ conda update --all --yes
2121
# required to use conda develop
2222
conda install -y conda-build
2323

24+
# Use faster conda solver
25+
conda install -n base conda-libmamba-solver
26+
conda config --set experimental_solver libmamba
27+
2428
# install other frameworks if asked for and make sure this is before pytorch
2529
if [[ $FRAMEWORKS == true ]]; then
2630
pip install pytext-nlp
@@ -35,10 +39,13 @@ else
3539
fi
3640

3741
# install other deps
38-
conda install -y numpy sphinx pytest flake8 ipywidgets ipython scikit-learn parameterized
39-
conda install -y -c conda-forge matplotlib pytest-cov sphinx-autodoc-typehints mypy flask flask-compress
42+
# conda install -y numpy sphinx pytest flake8 ipywidgets ipython scikit-learn parameterized
43+
# conda install -y -c conda-forge matplotlib pytest-cov sphinx-autodoc-typehints mypy flask flask-compress
44+
conda install -y pytest flake8 ipywidgets ipython scikit-learn parameterized
45+
conda install -y -c conda-forge matplotlib pytest-cov mypy flask flask-compress
46+
4047
# deps not available in conda
41-
pip install sphinxcontrib-katex
48+
# pip install sphinxcontrib-katex
4249

4350
# install node/yarn for insights build
4451
conda install -y -c conda-forge yarn

0 commit comments

Comments
 (0)