Skip to content

Conversation

@pschmutz
Copy link

Description

The links for APPO were referencing the PPO paper. I updated them to link to the IMPACT paper

@pschmutz pschmutz requested review from a team as code owners November 24, 2025 15:48
Copy link
Member

@pseudo-rnd-thoughts pseudo-rnd-thoughts left a comment

Choose a reason for hiding this comment

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

Thanks for the PR

@pseudo-rnd-thoughts pseudo-rnd-thoughts added rllib RLlib related issues go add ONLY when ready to merge, run all tests labels Nov 24, 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 correctly updates the paper reference for the APPO algorithm in the documentation. The previous link pointed to the PPO paper, and this change corrects it to the IMPACT paper, which is the original publication for APPO. The change is accurate and improves the documentation.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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.

2 participants