CNN Architecture
Opened this issue · 0 comments
araffin commented
Hello,
I should have written this issue when we noticed it a while ago but the architecture of the CNN does not match the Nature CNN one (I assume that was the goal), the last layer should have 64 channels too.
This repo:
pytorch-a2c-ppo-acktr-gail/a2c_ppo_acktr/model.py
Lines 176 to 180 in f60ac80
SB2 repo (following OpenAI Baselines repo):
https://github.com/hill-a/stable-baselines/blob/a4efff01ca678bcceee3eb21801c410612df209f/stable_baselines/common/policies.py#L16-L29
or in the SB3 repo:
https://github.com/DLR-RM/stable-baselines3/blob/88e1be9ff5e04b7688efa44951f845b7daf5717f/stable_baselines3/common/torch_layers.py#L76-L84