Skip to content

dr mario env#530

Merged
jsuarez5341 merged 2 commits intoPufferAI:4.0from
doofenshirmtz:drmario-env
Apr 17, 2026
Merged

dr mario env#530
jsuarez5341 merged 2 commits intoPufferAI:4.0from
doofenshirmtz:drmario-env

Conversation

@doofenshirmtz
Copy link
Copy Markdown
Contributor

rewards configs were modified but perfs dont exist save me jospeh

@jsuarez5341
Copy link
Copy Markdown
Contributor

Solid first PR, reviewed on stream. Minor style stuff:

  • don't newline curlybraces
  • Learn inverted if pattern / early return
  • Don't 1-line continue statements
  • Semantic grouping for ops (a + b*c not a + b * c)
  • When you get really long conditionals because the vars have long names, rename or temp var to shorten

I will help debug training on stream. PR some small fixes with the above and then yourself + collaborator to the contributors list on the website repo. Have him add that dino env too, I missed that one!

@jsuarez5341 jsuarez5341 merged commit 534fd82 into PufferAI:4.0 Apr 17, 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.

2 participants