Skip to content

Cache also .venv for mujoco_ros2_control#812

Merged
christophfroehlich merged 1 commit into
masterfrom
cache-venv
May 25, 2026
Merged

Cache also .venv for mujoco_ros2_control#812
christophfroehlich merged 1 commit into
masterfrom
cache-venv

Conversation

@christophfroehlich
Copy link
Copy Markdown
Member

Could improve ros-controls/mujoco_ros2_control#193
But: it needs to succeed once on the default branch to have a successful cache store, which then can be restored the next time:

- name: Restore ccache folder
uses: actions/cache/restore@v5
with:
path: ${{ env.CCACHE_DIR }}
key: ccache-${{ env.CACHE_PREFIX }}-${{ github.sha }}-${{ github.run_id }}
restore-keys: |
ccache-${{ env.CACHE_PREFIX }}-${{ github.sha }}
ccache-${{ env.CACHE_PREFIX }}

👀 @saikishor maybe we have to increase the timeout to let it be installed for the first time in every installation, it even fails in my local devcontainer:

3/6 Test #3: test_robot_launch_mjcf_generation_test.py ..........................***Timeout  75.36 sec

@christophfroehlich christophfroehlich merged commit 9d1d8e9 into master May 25, 2026
29 of 45 checks passed
@christophfroehlich christophfroehlich deleted the cache-venv branch May 25, 2026 16:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant