examples(teleop_ros2): package under isaacteleop_examples

[jiwenc-nv]

Description

Part of #985. Stacked on #1038.

Fifteen modules imported their siblings as top-level modules — constants, messages, geometry, assets, session_config and the rest — which resolved only because the invoked script's own directory landed on sys.path. That broke under python -m, broke when copied into another project, and claimed an unusually generic set of names. This moves the tree to the layout in examples/README.md.

Two workarounds propping this up are deleted, which is the evidence the packaging is real rather than relocated:

• pythonpath = ["."] in the pyproject
• the example source directory on the ctest PYTHONPATH

The tests reach the in-tree source through a conftest.py instead, mirroring mujoco_xr's, so a bare pytest in that directory works for the first time. Six test import lines change; no assertion does.

The node keeps its filename and gains a __main__.py, so the container entry point becomes python -m isaacteleop_examples.teleop_ros2 and still forwards --ros-args. The install tree drops a level, so the Dockerfile WORKDIR follows, as do the three invocations in build-ubuntu.yml.

Type of change

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)
• Documentation update

Testing

x86_64 / Ubuntu, Python 3.12. Wheel ships 15 modules under isaacteleop_examples/teleop_ros2/ with no namespace __init__.py. Editable-installed into a clean venv, the ROS-free modules (constants, tensor_group_helpers) import with the CWD outside the repo; import constants fails, so the flat names are gone rather than relocated. The new conftest.py was checked to resolve to examples/teleop_ros2/python with the package present and no namespace __init__.py. All test files compile. pre-commit clean.

Not run: ctest -L teleop_ros2 — no ROS 2 on this host, so geometry_msgs and rclpy are unavailable. That suite and the container entry point are what CI should be trusted for here; please treat the CI run as the gate rather than my local checks.

Checklist

• I have read and understood the contribution guidelines
• I have run the linter and formatter with SKIP=check-copyright-year pre-commit run --all-files
• I have made corresponding changes to the documentation
• I have added tests that prove my fix/feature works (or explained why not)
• I have signed off all my commits (git commit -s) per the DCO

_{No new tests: this is a move. The existing suite is the check — it exercises every moved module and passes with only its import lines changed.}

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Enterprise

Run ID: 2446ad5b-c732-45ab-8f8f-c81ee1a7e31e

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

---

_{Comment @coderabbitai help to get the list of available commands.}