Review of Existing Code · Issue #17 · DLR-RM/stable-baselines3 (original) (raw)

@araffin has done a great job creating this version of the library but has done it mostly solo, so much of the code has never been reviewed.

I suggest the other maintainers each take responsibility for reviewing a portion of the code. Rather than doing a traditional code review, since it's already committed I suggest we just make a PR with any changes we think should take place, or raise an issue for non-trivial proposals.

I think the priority is code that is used in multiple algorithms and/or defines the public API and which is new. This includes:

• common/base_class.py
• common/distributions.py
• common/policies.py
• common/type_aliases.py

Next would be the individual algorithms:

• A2C
• PPO
• SAC
• TD3

Also new parts of the documentation could use re-reading/confirming:

• Documentation

If this sounds like a good idea to others, then perhaps we can each claim a few entries on the above list and then start a PR for it? Also feel free to edit the post to break it up differently or to add files I've missed.