Update and simplify tutorial#383
Conversation
Greptile SummarySimplified and modernized the OpenEnv tutorial, reducing content from ~1260 lines to 220 lines while maintaining essential information. The new structure is more direct and focuses on practical getting-started content. Major changes:
Benefits:
Confidence Score: 5/5
Important Files Changed
Last reviewed commit: b8ec925 |
burtenshaw
left a comment
There was a problem hiding this comment.
This looks nice to me. But I'll let @init27 review the notebook as the author.
init27
left a comment
There was a problem hiding this comment.
Thanks very much for updating this @sergiopaniego!
Actually, I would request keeping the old style, since the notebook is designed to be an e2e onboarding for RL + OpenEnv.
Based on the 8-9 workshops we have done in-person, my experience is the audience generally still requires an onboard to RL so I would prefer keeping the concepts and onboarding in there please.
|
Thanks for the feedback @init27 @sergiopaniego Could we focus the PR on just the md docs example? I think there it makes sense to have a more concise page to bootstrap users' projects. |
burtenshaw
left a comment
There was a problem hiding this comment.
Just the docs changes look good. LGTM
Summary
This PR updates and simplifies the tutorial both the notebook and doc page.
The new structure tries to be more direct in showing the main features and links to additional content for more details.
The new tutorial needs to be run with updated envs since it uses
asyncso, although I have tested it using updated envs that pull OpenEnv from main, that would need to be updated for it to workCould depend on #293
Type of Change
Alignment Checklist
Before submitting, verify:
.claude/docs/PRINCIPLES.mdand this PR aligns with our principles.claude/docs/INVARIANTS.mdand no invariants are violated/pre-submit-pr(orbash .claude/hooks/lint.shand tests) and addressed all issuesRFC Status
Test Plan
Running the notebook! :)
Claude Code Review
"N/A"