SforAiDl / genrl

A PyTorch reinforcement learning library for generalizable and reproducible algorithm implementations with an aim to improve accessibility in RL

https://genrl.readthedocs.io

MIT License

404 stars 59 forks source link

LGTM alerts #343

Closed Sharad24 closed 3 years ago

Sharad24 commented 3 years ago

Closes #342

codecov[bot] commented 3 years ago

Codecov Report

Merging #343 into master will decrease coverage by 0.00%. The diff coverage is 25.00%.

@@            Coverage Diff             @@
##           master     #343      +/-   ##
==========================================
- Coverage   90.03%   90.03%   -0.01%     
==========================================
  Files          88       88              
  Lines        3694     3693       -1     
==========================================
- Hits         3326     3325       -1     
  Misses        368      368

Impacted Files	Coverage Δ
genrl/evolutionary/genetic_hyperparam.py	`0.00% <0.00%> (ø)`
genrl/trainers/classical.py	`86.00% <100.00%> (ø)`
genrl/agents/bandits/multiarmed/epsgreedy.py	`96.29% <0.00%> (-3.71%)`	:arrow_down:

lgtm-com[bot] commented 3 years ago

This pull request introduces 1 alert and fixes 4 when merging 78a2bab52c813b3db81b994357db1483b9a650f1 into 31c9b2efc9976e6c4b2efd9a75068c3ab7b790f6 - view on LGTM.com

new alerts:

1 for Variable defined multiple times

fixed alerts:

1 for Unused local variable
1 for Unused import
1 for Suspicious unused loop iteration variable
1 for Variable defined multiple times

lgtm-com[bot] commented 3 years ago

This pull request fixes 4 alerts when merging 947ddf43d7730511740a3daf15f3753658ccffe2 into a0bb994a6d7167ccf0cf67c61b716240fe9b32ba - view on LGTM.com

fixed alerts:

1 for Unused local variable
1 for Unused import
1 for Suspicious unused loop iteration variable
1 for Variable defined multiple times