opendilab / DI-engine

OpenDILab Decision AI Engine
https://di-engine-docs.readthedocs.io
Apache License 2.0
2.79k stars 348 forks source link

polish(pu): polish comments in a2c/bcq/fqf/ibc policy #768

Closed puyuan1996 closed 5 months ago

puyuan1996 commented 5 months ago

Description

Related Issue

TODO

Check List

codecov[bot] commented 5 months ago

Codecov Report

Attention: 4 lines in your changes are missing coverage. Please review.

Comparison is base (a57bc30) 76.78% compared to head (1e07862) 76.79%.

Files Patch % Lines
ding/policy/bcq.py 66.66% 4 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #768 +/- ## ======================================= Coverage 76.78% 76.79% ======================================= Files 676 676 Lines 54431 54427 -4 ======================================= - Hits 41797 41796 -1 + Misses 12634 12631 -3 ``` | [Flag](https://app.codecov.io/gh/opendilab/DI-engine/pull/768/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=opendilab) | Coverage Δ | | |---|---|---| | [unittests](https://app.codecov.io/gh/opendilab/DI-engine/pull/768/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=opendilab) | `76.79% <87.87%> (+<0.01%)` | :arrow_up: | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=opendilab#carryforward-flags-in-the-pull-request-comment) to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.