thu-ml/tianshou

Re-examine the need of utils.net.common.DataParallelNet

MischaPanch opened this issue · 0 comments

It does essentially nothing (just casting to tensor) and obscures interfaces. Can it be removed? Only used in ppo example for some reason

@opcode81 related to our discussion about interface of forward becoming a tensor instead of array | tensor. Wanna look into it?