Skip to content

[RLlib] remove MLAgents dependency#59524

Merged
simonsays1980 merged 10 commits intomasterfrom
kk/drop-mlagents-dependency
Dec 19, 2025
Merged

[RLlib] remove MLAgents dependency#59524
simonsays1980 merged 10 commits intomasterfrom
kk/drop-mlagents-dependency

Conversation

@kamil-kaczmarek
Copy link
Contributor

Description

This pull request removes Unity3D-based environments (mlagents and mlagents_envs) from RLlib, including dependencies, code, documentation, and related test requirements.

The main goal is to clean up the requirements dependency.

Signed-off-by: Kamil Kaczmarek <kamil@anyscale.com>
Signed-off-by: Kamil Kaczmarek <kamil@anyscale.com>
Signed-off-by: Kamil Kaczmarek <kamil@anyscale.com>
Signed-off-by: Kamil Kaczmarek <kamil@anyscale.com>
Signed-off-by: Kamil Kaczmarek <kamil@anyscale.com>
Signed-off-by: Kamil Kaczmarek <kamil@anyscale.com>
@kamil-kaczmarek kamil-kaczmarek self-assigned this Dec 18, 2025
@kamil-kaczmarek kamil-kaczmarek requested a review from a team as a code owner December 18, 2025 01:22
@kamil-kaczmarek kamil-kaczmarek added the dependencies Pull requests that update a dependency file label Dec 18, 2025
@kamil-kaczmarek kamil-kaczmarek requested a review from a team as a code owner December 18, 2025 01:22
@kamil-kaczmarek kamil-kaczmarek added the rllib RLlib related issues label Dec 18, 2025
Copy link
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request effectively removes the mlagents and mlagents_envs dependencies from RLlib. The changes are comprehensive, covering not only the package requirements but also the associated code for Unity3D environment wrappers, example scripts, documentation, and CI configurations. The removals are clean and consistent with the goal of reducing dependency clutter. The code base is now cleaner and has one less external dependency to maintain. The changes look good and I approve of this cleanup.

@kamil-kaczmarek kamil-kaczmarek changed the title Kk/drop mlagents dependency [RLlib] remove MLAgents dependency Dec 18, 2025
@kamil-kaczmarek kamil-kaczmarek added the go add ONLY when ready to merge, run all tests label Dec 18, 2025
Copy link
Contributor

@simonsays1980 simonsays1980 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Thanks for the PR @kamil-kaczmarek! Before merging let's discuss today in the meeting - this would remove a guideline for users who are working with the new stack on UnityEngine.

@simonsays1980 simonsays1980 enabled auto-merge (squash) December 18, 2025 21:14
@github-actions github-actions bot disabled auto-merge December 18, 2025 23:36
@simonsays1980 simonsays1980 merged commit 63ad404 into master Dec 19, 2025
6 checks passed
@simonsays1980 simonsays1980 deleted the kk/drop-mlagents-dependency branch December 19, 2025 10:39
Yicheng-Lu-llll pushed a commit to Yicheng-Lu-llll/ray that referenced this pull request Dec 22, 2025
## Description
This pull request removes Unity3D-based environments (`mlagents` and
`mlagents_envs`) from RLlib, including dependencies, code,
documentation, and related test requirements.

The main goal is to clean up the requirements dependency.

---------

Signed-off-by: Kamil Kaczmarek <kamil@anyscale.com>
AYou0207 pushed a commit to AYou0207/ray that referenced this pull request Jan 13, 2026
## Description
This pull request removes Unity3D-based environments (`mlagents` and
`mlagents_envs`) from RLlib, including dependencies, code,
documentation, and related test requirements.

The main goal is to clean up the requirements dependency.

---------

Signed-off-by: Kamil Kaczmarek <kamil@anyscale.com>
Signed-off-by: jasonwrwang <jasonwrwang@tencent.com>
lee1258561 pushed a commit to pinterest/ray that referenced this pull request Feb 3, 2026
## Description
This pull request removes Unity3D-based environments (`mlagents` and
`mlagents_envs`) from RLlib, including dependencies, code,
documentation, and related test requirements.

The main goal is to clean up the requirements dependency.

---------

Signed-off-by: Kamil Kaczmarek <kamil@anyscale.com>
peterxcli pushed a commit to peterxcli/ray that referenced this pull request Feb 25, 2026
## Description
This pull request removes Unity3D-based environments (`mlagents` and
`mlagents_envs`) from RLlib, including dependencies, code,
documentation, and related test requirements.

The main goal is to clean up the requirements dependency.

---------

Signed-off-by: Kamil Kaczmarek <kamil@anyscale.com>
Signed-off-by: peterxcli <peterxcli@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file go add ONLY when ready to merge, run all tests rllib RLlib related issues

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants