Codecov Report

:exclamation: No coverage uploaded for pull request base (main@960a652). Click here to learn what that means. The diff coverage is n/a.

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

@@           Coverage Diff           @@
##             main      #47   +/-   ##
=======================================
  Coverage        ?   36.36%           
=======================================
  Files           ?       53           
  Lines           ?     4314           
  Branches        ?        0           
=======================================
  Hits            ?     1569           
  Misses          ?     2745           
  Partials        ?        0

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

lgtm-com[bot] commented 2 years ago

This pull request introduces 9 alerts and fixes 21 when merging e8ff52117918c2577e2f77d66c7730e9ed765af2 into 960a652d74fc44b58f531d7b89222b8c8f28fbd1 - view on LGTM.com

new alerts:

9 for Unused import

fixed alerts:

8 for Unused import
3 for Wrong name for an argument in a call
3 for Unused local variable
2 for Signature mismatch in overriding method
2 for Missing call to `__init__` during object initialization
1 for Variable defined multiple times
1 for Wrong number of arguments in a call
1 for Wrong number of arguments in a class instantiation

Heads-up: LGTM.com's PR analysis will be disabled on the 5th of December, and LGTM.com will be shut down ⏻ completely on the 16th of December 2022. Please enable GitHub code scanning, which uses the same CodeQL engine :gear: that powers LGTM.com. For more information, please check out our post on the GitHub blog.

sjtu-marl / malib

Test cases #47

Codecov Report

1 for Wrong number of arguments in a class instantiation