Skip to content

chore: update news for scaffoldings#1236

Merged
garrett4wade merged 1 commit into
mainfrom
fw/news
Apr 23, 2026
Merged

chore: update news for scaffoldings#1236
garrett4wade merged 1 commit into
mainfrom
fw/news

Conversation

@garrett4wade

Copy link
Copy Markdown
Collaborator

No description provided.

@garrett4wade
garrett4wade merged commit 9fd34dc into main Apr 23, 2026
1 check passed
@garrett4wade
garrett4wade deleted the fw/news branch April 23, 2026 07:50

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

Copy link
Copy Markdown
Contributor

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 adds a new news entry to the README regarding the integration with Scaffolding for agentic RL training and reorganizes the 'Previous Releases' section. Feedback was provided to maintain date format consistency, correct the framework's name, and improve readability by reducing word repetition.

Comment thread README.md
Comment on lines +43 to +51
**\[2026-04-23\]** 🚀 We’re excited to release our integration with
[Scaffoldings](https://github.com/NVIDIA/TensorRT-LLM/tree/main/tensorrt_llm/scaffolding)
for agentic RL training - now live in our
[examples](https://github.com/inclusionAI/AReaL/tree/main/examples/scaffolding)! Huge
shoutout to @narutolhy and @WeiHaocheng for making this happen 🙌. The modular design of
the Scaffoldings enables it to achieve a thorough decoupling of agent execution, reward
calculation, and trajectory acquisition. This enables developers to reuse existing
modules when implementing an agentic RL method, allowing them to focus on their own
innovative modules.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

medium

The new news entry has a few minor issues with consistency and phrasing:

  1. Date Format: The date 2026-04-23 uses hyphens, whereas all other entries in the News section use slashes (e.g., 2026/04/18). It should be updated to 2026/04/23 for consistency.
  2. Terminology: The framework is referred to as "Scaffolding" in the linked repository and the local directory. "Scaffoldings" (plural) is slightly inconsistent with the official naming.
  3. Clarity: The word "enables" is used twice in consecutive sentences. Replacing one instance improves readability.
Suggested change
**\[2026-04-23\]** 🚀 We’re excited to release our integration with
[Scaffoldings](https://github.com/NVIDIA/TensorRT-LLM/tree/main/tensorrt_llm/scaffolding)
for agentic RL training - now live in our
[examples](https://github.com/inclusionAI/AReaL/tree/main/examples/scaffolding)! Huge
shoutout to @narutolhy and @WeiHaocheng for making this happen 🙌. The modular design of
the Scaffoldings enables it to achieve a thorough decoupling of agent execution, reward
calculation, and trajectory acquisition. This enables developers to reuse existing
modules when implementing an agentic RL method, allowing them to focus on their own
innovative modules.
**\[2026/04/23\]** 🚀 We’re excited to release our integration with
[Scaffolding](https://github.com/NVIDIA/TensorRT-LLM/tree/main/tensorrt_llm/scaffolding)
for agentic RL training - now live in our
[examples](https://github.com/inclusionAI/AReaL/tree/main/examples/scaffolding)! Huge
shoutout to @narutolhy and @WeiHaocheng for making this happen 🙌. The modular design of
Scaffolding enables a thorough decoupling of agent execution, reward
calculation, and trajectory acquisition. This allows developers to reuse existing
modules when implementing an agentic RL method, focusing on their own
innovative modules.

SathyaGnanakumar pushed a commit to danielkiely/AReaL that referenced this pull request Apr 29, 2026
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.

1 participant