enhancementgood first issue
Métriques du dépôt
- Stars
- (7 121 stars)
- Métriques de merge PR
- (Aucune PR mergée en 30 j)
Description
There are some ways in which reward/return/value normalization could be improved. But one most drastic thing is the following:
Currently PGPolicy instantiates self.ret_rms = RunningMeanStd(), and RunningMeanStd has a default value of clip_max=10. This cannot be adjusted by users! (except through monkey-patching, ofc)
This might work well for some standard envs, but the clipping value is arbitrary and making it non-configurable is a major hinderance for users, who are most probably not aware of this.
Generally, how to best normalize stuff in RL is an active discussion and normalization can play an important role in performance. I believe tianshou should be extended to accomodate various normalization strategies