Skip to content

Commit

Permalink
[CI] Fix olddeps dm_control
Browse files Browse the repository at this point in the history
ghstack-source-id: 750edcb8cd6b17167f77fb7c9ebd538608cfbde6
Pull Request resolved: #2734
  • Loading branch information
vmoens committed Feb 3, 2025
1 parent dbc8e2e commit 3ac6127
Show file tree
Hide file tree
Showing 3 changed files with 9 additions and 4 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ channels:
- pytorch
- defaults
dependencies:
- pip
- pip<25.0
- protobuf
- pip:
- hypothesis
Expand All @@ -21,7 +21,6 @@ dependencies:
- pyyaml
- scipy
- hydra-core
- dm_control -e git+https://github.com/deepmind/dm_control.git@c053360edea6170acfd9c8f65446703307d9d352#egg={dm_control}
- mujoco
- patchelf
- pyopengl==3.1.4
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@ export MKL_THREADING_LAYER=GNU
export BATCHED_PIPE_TIMEOUT=60

python .github/unittest/helpers/coverage_run_parallel.py -m pytest test/smoke_test.py -v --durations 200
python .github/unittest/helpers/coverage_run_parallel.py -m pytest test/smoke_test_deps.py -v --durations 200 -k 'test_gym or test_dm_control_pixels or test_dm_control'
python .github/unittest/helpers/coverage_run_parallel.py -m pytest test/smoke_test_deps.py -v --durations 200 -k 'test_gym'

export DISPLAY=':99.0'
Xvfb :99 -screen 0 1400x900x24 > /dev/null 2>&1 &
Expand Down
8 changes: 7 additions & 1 deletion .github/unittest/linux_olddeps/scripts_gym_0_13/setup_env.sh
Original file line number Diff line number Diff line change
Expand Up @@ -62,7 +62,13 @@ cp mjkey.txt mujoco_py/binaries/
pip install -e .
cd ..

#cd $this_dir
# OLD DM_CONTROL NOT SUPPORTED ANYMORE
# # Install dm_control
# git clone https://github.com/deepmind/dm_control
# cd dm_control
# git checkout c053360edea6170acfd9c8f65446703307d9d352
# pip install -e .
# cd ..

# 4. Install Conda dependencies
printf "* Installing dependencies (except PyTorch)\n"
Expand Down

0 comments on commit 3ac6127

Please sign in to comment.