facebookresearch / hanabi_SAD

Simplified Action Decoder for Deep Multi-Agent Reinforcement Learning
Other
96 stars 35 forks source link

num_ff_layer #23

Closed KevinJeon closed 3 years ago

KevinJeon commented 3 years ago

Is it right num_ff_layer in R2D2Net instead of 'num_fc_layer'?

hengyuan-hu commented 3 years ago

num_ff_layer refers to number of feed-forward layers, which in our case is indeed fully connected layers.

hengyuan-hu commented 3 years ago

Ah, now I understand what you mean. Yes, the num_ff_layer in R2D2Net should be changed to num_fc_layer.

KevinJeon commented 3 years ago

I uploaded it because I wasn't sure if num_ff_layer was exactly pointing to num_fc_layer during changing the code. Thank you for your answer :)