[RLlib] LINT: Enable ruff imports for offline/, tests/, callbacks/ and env/ in rllib

[czgdp1807]

Why are these changes needed?

Sixth split of #56416

Related issue number

Checks

• I've signed off every commit(by using the -s flag, i.e., git commit -s) in this PR.
• I've run scripts/format.sh to lint the changes in this PR.
• I've included any doc changes needed for https://docs.ray.io/en/master/.
  • I've added any new APIs to the API Reference. For example, if I added a
    method in Tune, I've added it in doc/source/tune/api/ under the
    corresponding .rst file.
• I've made sure the tests are passing. Note that there might be a few flaky tests, see the recent failures at https://flakey-tests.ray.io/
• Testing Strategy
  • Unit tests
  • Release tests
  • This PR is not tested :(

---

Note

Updates Ruff config to target specific rllib subpaths and performs widespread import reordering/normalization across callbacks, env, offline, and tests (no functional changes).

• Config:
  • Update pyproject.toml Ruff per-file-ignores from rllib/* to specific subpaths (e.g., rllib/{algorithms,core,examples,...}) for finer-grained linting.
• RLlib (callbacks, env, offline, tests):
  • Reorder and consolidate imports, normalize typing/import ordering, and adjust intra-package imports (e.g., combined imports, moved TYPE_CHECKING blocks).
  • Minor all/export and import path cleanups in __init__ and wrappers.
  • Test files: move imports, tidy main blocks; no behavioral logic modified.

^{Written by Cursor Bugbot for commit a6af7a4. This will update automatically on new commits. Configure here.}

[gemini-code-assist]

Code Review

This pull request enables ruff for import sorting on offline/, tests/, callbacks/, and env/ directories within rllib. The changes consist of updating pyproject.toml to remove these directories from the isort ignore list and applying the corresponding import reordering across numerous files in these directories. The changes are purely stylistic and align with the project's linting standards. The import reordering has been applied correctly across all modified files. The PR is a good step towards standardizing the codebase.

[kamil-kaczmarek]

LGTM! Great work

[pseudo-rnd-thoughts]

@czgdp1807 apologies, I was trying to fix the merge conflicts through the github web ui however it doesn't include the commit sign off that DCO requires. We can merge this PR last of your linting PRs but it will require you to rebase the PR to resolve this issue

[czgdp1807]

@pseudo-rnd-thoughts @kamil-kaczmarek I have merged master and resolved the conflicts. Should pass the tests now.

[pseudo-rnd-thoughts]

@czgdp1807 could you resolve them sadly. I suspect that we'll need to wait to premerge to pass then merge then fix the new conflicts on another one of the PRs until they're all complete

[czgdp1807]

No problems @pseudo-rnd-thoughts. This is expected to happen. Resolving them.

[czgdp1807]

@pseudo-rnd-thoughts Done.

[pseudo-rnd-thoughts]

@czgdp1807 There is a problem in the CI thats unrelated to your PR that were trying to actively fix in #57067

I'll ping you when the CI problem is resolved and if there is anything we need you to do

[github-actions]

This pull request has been automatically marked as stale because it has not had
any activity for 14 days. It will be closed in another 14 days if no further activity occurs.
Thank you for your contributions.

You can always ask for help on our discussion forum or Ray's public slack channel.

If you'd like to keep this open, just leave any comment, and the stale label will be removed.

[kamil-kaczmarek]

unstale