Skip to content

Fix examples CI without optional RLlib dependencies - #339

Open
divyanshu-iitian wants to merge 1 commit into
google-deepmind:mainfrom
divyanshu-iitian:fix/optional-rllib-test-deps
Open

Fix examples CI without optional RLlib dependencies#339
divyanshu-iitian wants to merge 1 commit into
google-deepmind:mainfrom
divyanshu-iitian:fix/optional-rllib-test-deps

Conversation

@divyanshu-iitian

Copy link
Copy Markdown

Summary

  • collect RLlib tests normally and skip them at class level when the optional Gymnasium/RLlib extras are unavailable
  • defer optional imports until skipped tests actually run, avoiding collection failures
  • scope lint and type-check import diagnostics in RLlib-only modules to their intentionally optional dependencies

This keeps the required examples workflow successful without making the large RLlib dependency set part of the core installation.

Fixes #316

Testing

  • python -m pytest examples/rllib -q: 6 skipped, exit code 0
  • pylint --errors-only examples/rllib: passed

@divyanshu-iitian
divyanshu-iitian force-pushed the fix/optional-rllib-test-deps branch from bb63250 to cda3a38 Compare August 10, 2026 05:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Required test-examples CI check fails for RLlib examples due to missing optional dependencies

1 participant