Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Update cleanrl-supported-papers-projects.md #316

Merged
merged 1 commit into from
Nov 8, 2022

Conversation

masud99r
Copy link
Collaborator

@masud99r masud99r commented Nov 8, 2022

Adding a paper that usages CleanRL for implementation.

Description

Types of changes

  • Bug fix
  • New feature
  • New algorithm
  • Documentation

Checklist:

  • I've read the CONTRIBUTION guide (required).
  • I have ensured pre-commit run --all-files passes (required).
  • I have updated the documentation and previewed the changes via mkdocs serve.
  • I have updated the tests accordingly (if applicable).

If you are adding new algorithms or your change could result in performance difference, you may need to (re-)run tracked experiments. See #137 as an example PR.

  • I have contacted vwxyzjn to obtain access to the openrlbenchmark W&B team (required).
  • I have tracked applicable experiments in openrlbenchmark/cleanrl with --capture-video flag toggled on (required).
  • I have added additional documentation and previewed the changes via mkdocs serve.
    • I have explained note-worthy implementation details.
    • I have explained the logged metrics.
    • I have added links to the original paper and related papers (if applicable).
    • I have added links to the PR related to the algorithm.
    • I have created a table comparing my results against those from reputable sources (i.e., the original paper or other reference implementation).
    • I have added the learning curves (in PNG format with width=500 and height=300).
    • I have added links to the tracked experiments.
    • I have updated the overview sections at the docs and the repo
  • I have updated the tests accordingly (if applicable).

Adding a paper that usages CleanRL for implementation.
@vercel
Copy link

vercel bot commented Nov 8, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
cleanrl ❌ Failed (Inspect) Nov 8, 2022 at 7:29PM (UTC)

@vwxyzjn vwxyzjn changed the base branch from master to 1.0-release November 8, 2022 19:39
@vwxyzjn vwxyzjn changed the base branch from 1.0-release to master November 8, 2022 19:39
@vwxyzjn
Copy link
Owner

vwxyzjn commented Nov 8, 2022

Thanks for the addition @masud99r. Docs build at vercel currently break because of #315. Merging now.

@vwxyzjn vwxyzjn merged commit 19a0907 into vwxyzjn:master Nov 8, 2022
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants