[jackiectl2]

Include traj_len - sub_traj_len as a possible start index so every complete sub-trajectory, including the terminal window, can be sampled.

Fixes #4708.

Validation

6 focused TestSubSampler tests passed; both window-coverage regressions fail without the fix. CPU torch 2.14.1 and TensorDict 0.14.2; the optional TorchRL C++ extension is not built. All changed-file pre-commit hooks passed under Python 3.11.

Full trainer suite and GPU training not run.

[pytorch-bot]

🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/rl/4709

• 📄 Preview Python docs built from this PR

Note: Links to docs will display an error until the docs builds have been completed.

⚠️ 16 Awaiting Approval

As of commit 9192b54 with merge base a711ef9 ():

AWAITING APPROVAL - The following workflows need approval before CI can run:

• Build Aarch64 Linux Wheels (gh)
• Build Linux Wheels (gh)
• Build M1 Wheels (gh)
• Build Windows Wheels (gh)
• Continuous Benchmark (PR) (gh)
• Examples Tests on Linux (gh)
• Generate documentation (gh)
• Habitat Tests on Linux (gh)
• Libs Tests on Linux (gh)
• Lint (gh)
• LLM Tests on Linux (gh)
• Push Binary Nightly (gh)
• SOTA Tests on Linux (gh)
• Tutorials Tests on Linux (gh)
• Unit-tests on Linux (gh)
• Validate Test Partitioning (gh)

This comment was automatically generated by Dr. CI and updates every 15 minutes.