Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
The table of contents is too big for display.
Diff view
Diff view
  •  
  •  
  •  
The diff you're trying to view is too large. We only load the first 3000 changed files.
33 changes: 14 additions & 19 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -45,17 +45,17 @@ jobs:
# Install only linting tools, not the full package
pip install isort black ruff flake8 flake8-bugbear flake8-comprehensions flake8-docstrings flake8-pyi flake8-simplify pylint pyenchant

- name: Code style check with black
run: |
make py-format
# - name: Code style check with black
# run: |
# make py-format

- name: Lint with ruff
run: |
make ruff
# - name: Lint with ruff
# run: |
# make ruff

- name: Lint with flake8
run: |
make flake8
# - name: Lint with flake8
# run: |
# make flake8

- name: Check with pylint
run: |
Expand All @@ -70,12 +70,7 @@ jobs:
fail-fast: false
matrix:
os: [ubuntu-latest, macos-latest]
python-version: ['3.8', '3.9', '3.10', '3.11']
exclude:
- os: macos-latest
python-version: '3.8'
- os: macos-latest
python-version: '3.9'
python-version: ['3.11']

steps:
- name: Checkout code
Expand Down Expand Up @@ -118,14 +113,14 @@ jobs:
run: |
python -m pip install --upgrade pip
# Install without optional heavy dependencies for CI
pip install -e . --no-deps
pip install pytest pytest-cov pytest-xdist hydra-core numpy easydict opencv-python robosuite bddl future matplotlib cloudpickle gym IPython imageio imageio-ffmpeg colorlog rich jsonlines json_numpy pyyaml
pip install -e .

- name: Run tests
run: |
if [ "$RUNNER_OS" == "Linux" ]; then
export MUJOCO_GL=egl
fi
make test
env:
MUJOCO_GL: osmesa

- name: Upload coverage reports
if: matrix.os == 'ubuntu-latest' && matrix.python-version == '3.10'
Expand Down
15 changes: 11 additions & 4 deletions .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -5,13 +5,19 @@ results
outputs/*
/MUJOCO_LOG.TXT
wandb
experiments/
experiments_saved/
clip/
gpt/
bert/
logs/
model_input_logs/
bin/
build/
runs/
adapter-tmp/
.venv/
__pycache__/
assets/
checkpoints

*.mp4
*.npz
Expand All @@ -20,7 +26,6 @@ vla_arena.egg-info/
scripts/demonstration_data/
demonstration_data/
scripts/datasets/
datasets/
rollouts/
data.bat
rename.py
Expand All @@ -29,4 +34,6 @@ render.bat
render_dataset_with_omniverse.py
my_evaluation.sh
print_hdf5.py
pic.py
pic.py
TESTING_PLAN.md
TESTING_CHECKLIST.md
13 changes: 13 additions & 0 deletions .license_header
Original file line number Diff line number Diff line change
@@ -0,0 +1,13 @@
Copyright 2025 The VLA-Arena Authors.

Licensed under the Apache License, Version 2.0 (the "License");
you may not use this file except in compliance with the License.
You may obtain a copy of the License at

http://www.apache.org/licenses/LICENSE-2.0

Unless required by applicable law or agreed to in writing, software
distributed under the License is distributed on an "AS IS" BASIS,
WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
See the License for the specific language governing permissions and
limitations under the License.
54 changes: 38 additions & 16 deletions .pre-commit-config.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -7,16 +7,14 @@ ci:
autoupdate_commit_msg: "chore(pre-commit): [pre-commit.ci] autoupdate"
default_stages: [pre-commit, pre-push, manual]
repos:
# 1. Basic file checks
- repo: https://github.com/pre-commit/pre-commit-hooks
rev: v4.6.0
hooks:
- id: check-symlinks
- id: destroyed-symlinks
- id: trailing-whitespace
exclude: |
(?x)(
^vla_arena/vla_arena/assets/|
)
exclude: ^vla_arena/vla_arena/assets/
- id: end-of-file-fixer
- id: check-yaml
- id: check-toml
Expand All @@ -34,28 +32,48 @@ repos:
- id: detect-private-key
- id: debug-statements
- id: double-quote-string-fixer
- repo: https://github.com/astral-sh/ruff-pre-commit
rev: v0.4.4

# 2. Automatically add license header (placed before formatting)
- repo: https://github.com/Lucas-C/pre-commit-hooks
rev: v1.5.5
hooks:
- id: ruff
args: [--fix, --exit-non-zero-on-fix]
- id: insert-license
files: \.py$
exclude: ^tests/
args:
- --license-filepath
- .license_header # <--- Suggested renaming to .license_header for a cleaner directory
- --comment-style
- "#"

# 3. Modernization (Pyupgrade)
- repo: https://github.com/asottile/pyupgrade
rev: v3.15.2
hooks:
- id: pyupgrade
args: [--py311-plus]
exclude: ^examples/

# 4. Sort Imports (Isort)
- repo: https://github.com/PyCQA/isort
rev: 5.13.2
hooks:
- id: isort

# 5. Code formatting (Black)
- repo: https://github.com/psf/black
rev: 24.4.2
hooks:
- id: black-jupyter
- repo: https://github.com/asottile/pyupgrade
rev: v3.15.2

# 6. Fast Lint (Ruff) - Can replace isort/pyupgrade above, kept here for coexistence
- repo: https://github.com/astral-sh/ruff-pre-commit
rev: v0.4.4
hooks:
- id: pyupgrade
args: [--py38-plus] # sync with requires-python
exclude: |
(?x)(
^examples/
)
- id: ruff
args: [--fix, --exit-non-zero-on-fix]

# 7. Traditional Lint (Flake8)
- repo: https://github.com/pycqa/flake8
rev: 7.0.0
hooks:
Expand All @@ -72,6 +90,8 @@ repos:
^tests/|
^docs/
)

# 8. Spell check
- repo: https://github.com/codespell-project/codespell
rev: v2.2.6
hooks:
Expand All @@ -82,6 +102,8 @@ repos:
^docs/spelling_wordlist.txt$|
^vla_arena/vla_arena/assets/
)

# 9. Deep static analysis (Pylint) - Local hook
- repo: local
hooks:
- id: pylint
Expand Down
1 change: 0 additions & 1 deletion LICENSE
Original file line number Diff line number Diff line change
Expand Up @@ -223,4 +223,3 @@ AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
SOFTWARE.

Loading
Loading