Skip to content

Conversation

@m-naumann
Copy link
Collaborator

No description provided.

EllingtonKirby and others added 11 commits June 3, 2025 10:50
Currently it is possible to unknowngly use two different seeds for the
PPO and dataloader. Normally if everyone was using 42 for the seed there
would be no effect, but we tested with varying seeds and had unintuitive
results, so I think matching the seeds here makes sense.

Co-authored-by: Ellington Kirby <ellingtonkirby@gmail.com>
* [FEATURE] cherry pick the json parsing update

* [TEST] adding a simple json parsing test

---------

Co-authored-by: Ellington Kirby <ellingtonkirby@gmail.com>
Co-authored-by: Ellington Kirby <ellingtonkirby@gmail.com>
* Update madrona to commit 4bda334 (from fork), which is uv compatible

* Use uv compatible build backend of madrona

* Switch to uv

* Update readme, include pyenv again
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.

5 participants