[ludgerpaehler]

Summary

Nine compiled Python files were tracked under koopmanrl/environments/__pycache__/ and koopmanrl/koopman_tensor/__pycache__/. Two of them were the bytecode of the cartpole environments, which have since been removed. They showed up as modified after every run and gave every clone stale bytecode. This PR stops tracking them and ignores compiled files in every directory. It also adds a pre-commit hook, so a compiled file that someone force-adds fails both locally and in CI.

Changes

• Remove the tracked .pyc files (git rm): 7 under koopmanrl/environments/__pycache__/ (including cartpole_control_env and continuous_cartpole) and 2 under koopmanrl/koopman_tensor/__pycache__/. Python regenerates them on import.
• .gitignore: the six per-directory <dir>/__pycache__/ entries are replaced by one general rule under the existing # Ignore PyCaches header. The old entries missed koopmanrl/koopman_tensor/ and observables/, which is how these files got in. The new rule is:

  __pycache__/
  *.py[cod]
  

  No other entries change.
• .pre-commit-config.yaml: a new repo: local hook, forbid-compiled-python (language: fail, files: '(\.py[cod]$|(^|/)__pycache__/)'). It rejects any .pyc, .pyo or .pyd file, and any file under a __pycache__/ directory, including one at the top level. .gitignore cannot stop git add -f, but this hook does. .github/workflows/lint.yml runs uv run pre-commit run --all-files, which checks the tracked files, so a compiled file that is committed anyway also fails the Lint workflow.
• Guides:
  • koopmanrl/environments/AGENTS.md: the bullet that warned about tracked .pyc files now says they are gitignored and rejected by the hook.
  • Root AGENTS.md: the hook is added to the list of pre-commit hooks, and compiled files are added to the list of ignored files.
  • docs/docs/developers/contributing.md: the hook is added to the list of hooks.

Test plan

• uv run pre-commit run --all-files passes on the branch. All hooks pass; forbid-compiled-python reports "no files to check".
• The hook works: I force-added a dummy koopmanrl/__pycache__/dummy.cpython-310.pyc. Both uv run pre-commit run --files <it> and --all-files then fail with "Compiled Python files (.pyc, .pyo, .pyd, pycache/) must not be committed". I removed the dummy afterwards.
• After uv run python -c "import koopmanrl.environments, koopmanrl.koopman_tensor.torch_tensor", git status is clean. git status --ignored lists the new __pycache__/ directories as ignored.
• Fresh-machine check on a git clone of the branch:
  • The clone contains no .pyc files and no __pycache__/ directories.
  • After uv sync, uv run python -c "import koopmanrl.environments" creates __pycache__/, and git status --ignored shows it as ignored (!!).
  • uv run pytest -q tests/test_koopman_tensor_regressors.py gives 17 passed. Its __pycache__/ directories under koopman_tensor/, observables/ and tests/ are also ignored.

🤖 Generated with Claude Code