Unverified Commit c627940a authored by Mateo Guaman Castro's avatar Mateo Guaman Castro Committed by GitHub

Adds wandb and uv env to dockerignore (#4027)

# Description

Currently, `.dockerignore` does not include `env_isaaclab` nor
`**/wandb/*` in the files to be ignored, which should be ignored for
cluster deployment since a) all Python packages are already part of the
docker image (if `env_isaaclab` is not included in `.dockerignore`, it
rsyncs over a 24GB dir), and b) local wandb logs should be ignored. This
PR adds `env_isaaclab` and `**/wandb/*` to `.dockerignore`.

Fixes #4026 

## Type of change

- Bug fix (non-breaking change which fixes an issue)

## Screenshots

Please attach before and after screenshots of the change if applicable.

Before:
<img width="692" height="199" alt="Screenshot from 2025-11-16 15-27-52"
src="https://github.com/user-attachments/assets/a53d2224-13ca-43df-9732-618b9adc0ac9"
/>

After:
<img width="692" height="182" alt="Screenshot from 2025-11-16 15-29-33"
src="https://github.com/user-attachments/assets/e2a9e7e3-5b8a-4566-8b5a-f1ee7ec57d3a"
/>

## Checklist

- [x] I have read and understood the [contribution
guidelines](https://isaac-sim.github.io/IsaacLab/main/source/refs/contributing.html)
- [x] I have run the [`pre-commit` checks](https://pre-commit.com/) with
`./isaaclab.sh --format`
- [ ] I have made corresponding changes to the documentation
- [ ] My changes generate no new warnings
- [ ] I have added tests that prove my fix is effective or that my
feature works
- [ ] I have updated the changelog and the corresponding version in the
extension's `config/extension.toml` file
- [x] I have added my name to the `CONTRIBUTORS.md` or my name already
exists there
parent 9cbf024e
......@@ -12,6 +12,7 @@ docs/
**/output/*
**/outputs/*
**/videos/*
**/wandb/*
*.tmp
# ignore docker
docker/cluster/exports/
......@@ -25,3 +26,5 @@ recordings/
_isaac_sim?
# Docker history
docker/.isaac-lab-docker-history
# ignore uv environment
env_isaaclab
......@@ -96,6 +96,7 @@ Guidelines for modifications:
* Lukas Fröhlich
* Manuel Schweiger
* Masoud Moghani
* Mateo Guaman Castro
* Maurice Rahme
* Michael Gussert
* Michael Noseworthy
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment