opendilab / DI-engine

OpenDILab Decision AI Engine. The Most Comprehensive Reinforcement Learning Framework B.P.
https://di-engine-docs.readthedocs.io
Apache License 2.0
2.82k stars 352 forks source link

polish(rjy): polish comments in wqmix/ngu/pg model #739

Closed nighood closed 8 months ago

nighood commented 9 months ago

Description

Related Issue

TODO

Check List

codecov[bot] commented 9 months ago

Codecov Report

Merging #739 (a9981fa) into main (817b215) will increase coverage by 0.13%. Report is 5 commits behind head on main. The diff coverage is n/a.

:exclamation: Current head a9981fa differs from pull request most recent head c385311. Consider uploading reports for the commit c385311 to get more accurate results

@@            Coverage Diff             @@
##             main     #739      +/-   ##
==========================================
+ Coverage   77.95%   78.09%   +0.13%     
==========================================
  Files         665      665              
  Lines       52179    52177       -2     
==========================================
+ Hits        40675    40746      +71     
+ Misses      11504    11431      -73     
Flag Coverage Δ
unittests 78.09% <ø> (+0.13%) :arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Coverage Δ
ding/model/template/ngu.py 78.88% <ø> (ø)
ding/model/template/pg.py 94.11% <ø> (ø)
ding/model/template/wqmix.py 96.77% <ø> (+0.98%) :arrow_up:

... and 15 files with indirect coverage changes