/
main_pylint_log.txt
969 lines (937 loc) · 59.9 KB
/
main_pylint_log.txt
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
249
250
251
252
253
254
255
256
257
258
259
260
261
262
263
264
265
266
267
268
269
270
271
272
273
274
275
276
277
278
279
280
281
282
283
284
285
286
287
288
289
290
291
292
293
294
295
296
297
298
299
300
301
302
303
304
305
306
307
308
309
310
311
312
313
314
315
316
317
318
319
320
321
322
323
324
325
326
327
328
329
330
331
332
333
334
335
336
337
338
339
340
341
342
343
344
345
346
347
348
349
350
351
352
353
354
355
356
357
358
359
360
361
362
363
364
365
366
367
368
369
370
371
372
373
374
375
376
377
378
379
380
381
382
383
384
385
386
387
388
389
390
391
392
393
394
395
396
397
398
399
400
401
402
403
404
405
406
407
408
409
410
411
412
413
414
415
416
417
418
419
420
421
422
423
424
425
426
427
428
429
430
431
432
433
434
435
436
437
438
439
440
441
442
443
444
445
446
447
448
449
450
451
452
453
454
455
456
457
458
459
460
461
462
463
464
465
466
467
468
469
470
471
472
473
474
475
476
477
478
479
480
481
482
483
484
485
486
487
488
489
490
491
492
493
494
495
496
497
498
499
500
501
502
503
504
505
506
507
508
509
510
511
512
513
514
515
516
517
518
519
520
521
522
523
524
525
526
527
528
529
530
531
532
533
534
535
536
537
538
539
540
541
542
543
544
545
546
547
548
549
550
551
552
553
554
555
556
557
558
559
560
561
562
563
564
565
566
567
568
569
570
571
572
573
574
575
576
577
578
579
580
581
582
583
584
585
586
587
588
589
590
591
592
593
594
595
596
597
598
599
600
601
602
603
604
605
606
607
608
609
610
611
612
613
614
615
616
617
618
619
620
621
622
623
624
625
626
627
628
629
630
631
632
633
634
635
636
637
638
639
640
641
642
643
644
645
646
647
648
649
650
651
652
653
654
655
656
657
658
659
660
661
662
663
664
665
666
667
668
669
670
671
672
673
674
675
676
677
678
679
680
681
682
683
684
685
686
687
688
689
690
691
692
693
694
695
696
697
698
699
700
701
702
703
704
705
706
707
708
709
710
711
712
713
714
715
716
717
718
719
720
721
722
723
724
725
726
727
728
729
730
731
732
733
734
735
736
737
738
739
740
741
742
743
744
745
746
747
748
749
750
751
752
753
754
755
756
757
758
759
760
761
762
763
764
765
766
767
768
769
770
771
772
773
774
775
776
777
778
779
780
781
782
783
784
785
786
787
788
789
790
791
792
793
794
795
796
797
798
799
800
801
802
803
804
805
806
807
808
809
810
811
812
813
814
815
816
817
818
819
820
821
822
823
824
825
826
827
828
829
830
831
832
833
834
835
836
837
838
839
840
841
842
843
844
845
846
847
848
849
850
851
852
853
854
855
856
857
858
859
860
861
862
863
864
865
866
867
868
869
870
871
872
873
874
875
876
877
878
879
880
881
882
883
884
885
886
887
888
889
890
891
892
893
894
895
896
897
898
899
900
901
902
903
904
905
906
907
908
909
910
911
912
913
914
915
916
917
918
919
920
921
922
923
924
925
926
927
928
929
930
931
932
933
934
935
936
937
938
939
940
941
942
943
944
945
946
947
948
949
950
951
952
953
954
955
956
957
958
959
960
961
962
963
964
965
966
967
968
************* Module conf
W: 73, 0: Redefining built-in 'copyright' (redefined-builtin)
C: 1, 0: Missing module docstring (missing-docstring)
R: 64, 4: Redefinition of source_suffix type from str to list (redefined-variable-type)
************* Module configuration
W: 29, 0: XXX Should be >= 10000 to be interesting "asymptotically" (fixme)
W: 41, 0: XXX to disable this optimization (fixme)
W: 44, 0: XXX Should be >= 10 to be stastically trustworthy (fixme)
W: 45, 0: XXX To profile the code, turn down parallel computing (fixme)
W: 52, 0: XXX To profile the code, turn down parallel computing (fixme)
W: 54, 0: XXX do not let this = False # To profile the code, turn down parallel computing (fixme)
W: 82, 0: XXX try different values for the learning rate for my aggregated bandit (fixme)
W: 94, 0: XXX try different values for time tau for the decreasing rate for my aggregated bandit (fixme)
W:100, 0: XXX do not let this = False if you want to test my Aggr policy (fixme)
W:104, 0: XXX do not let this = False (fixme)
W:119, 0: XXX This dictionary configures the experiments (fixme)
W:172, 0: XXX if using Exponential arms, gives klExp to KL-UCB-like policies! (fixme)
W:180, 0: XXX if using Gaussian arms, gives klGauss to KL-UCB-like policies! (fixme)
W:242, 0: XXX Efficient parameter-free Softmax (fixme)
W:334, 0: XXX Below the theoretically acceptable value! (fixme)
W:340, 0: XXX Below the theoretically acceptable value! (fixme)
W:346, 0: XXX Below the theoretically acceptable value! (fixme)
W:352, 0: XXX Below the theoretically acceptable value! (fixme)
W:419, 0: XXX Only test with fixed arms (fixme)
W:426, 0: XXX Only test with scenario 1 from [A.Beygelzimer, J.Langfor, L.Li et al, AISTATS 2011] (fixme)
W:427, 0: XXX remove after testing once (fixme)
W:434, 0: XXX If needed! (fixme)
W:440, 0: XXX old code to test different static learning rates, not any more (fixme)
W:441, 0: XXX to test between biased or unabiased estimators (fixme)
W:442, 0: XXX If needed! (fixme)
W:443, 0: XXX If needed! (fixme)
W:456, 0: XXX uncomment to give the value of horizon to have a better learning rate (fixme)
W: 24, 0: Wildcard import Policies (wildcard-import)
E: 26, 0: No name 'klucbBern' in module 'Policies.kullback' (no-name-in-module)
E: 26, 0: No name 'klucbExp' in module 'Policies.kullback' (no-name-in-module)
E: 26, 0: No name 'klucbGauss' in module 'Policies.kullback' (no-name-in-module)
C:427, 0: Import "from PoliciesMultiPlayers import Scenario1" should be placed at the top of the module (wrong-import-position)
W: 18, 0: Unused makeMeans imported from Arms (unused-import)
W: 20, 0: Unused Exponential imported from Arms.Exponential (unused-import)
W: 21, 0: Unused Gaussian imported from Arms.Gaussian (unused-import)
W: 22, 0: Unused Poisson imported from Arms.Poisson (unused-import)
W: 24, 0: Unused import UCBopt from wildcard import (unused-wildcard-import)
W: 24, 0: Unused import MOSS from wildcard import (unused-wildcard-import)
W: 24, 0: Unused import klUCBHPlus from wildcard import (unused-wildcard-import)
W: 24, 0: Unused import SoftMix from wildcard import (unused-wildcard-import)
W: 24, 0: Unused import Softmax from wildcard import (unused-wildcard-import)
W: 24, 0: Unused import TakeFixedArm from wildcard import (unused-wildcard-import)
W: 24, 0: Unused import UCBVtuned from wildcard import (unused-wildcard-import)
W: 24, 0: Unused import UCBV from wildcard import (unused-wildcard-import)
W: 24, 0: Unused import MEGA from wildcard import (unused-wildcard-import)
W: 24, 0: Unused import UCBplus from wildcard import (unused-wildcard-import)
W: 24, 0: Unused import Exp3WithHorizon from wildcard import (unused-wildcard-import)
W: 24, 0: Unused import SoftmaxWithHorizon from wildcard import (unused-wildcard-import)
W: 24, 0: Unused import Exp3Decreasing from wildcard import (unused-wildcard-import)
W: 24, 0: Unused import UCB from wildcard import (unused-wildcard-import)
W: 24, 0: Unused import UCBrandomInit from wildcard import (unused-wildcard-import)
W: 24, 0: Unused import UCBwrong from wildcard import (unused-wildcard-import)
W: 24, 0: Unused import AdBandits from wildcard import (unused-wildcard-import)
W: 24, 0: Unused import UniformOnSome from wildcard import (unused-wildcard-import)
W: 24, 0: Unused import EpsilonDecreasingMEGA from wildcard import (unused-wildcard-import)
W: 24, 0: Unused import Exp3SoftMix from wildcard import (unused-wildcard-import)
W: 24, 0: Unused import TakeRandomFixedArm from wildcard import (unused-wildcard-import)
W: 24, 0: Unused import EpsilonFirst from wildcard import (unused-wildcard-import)
W: 24, 0: Unused import EpsilonExpDecreasing from wildcard import (unused-wildcard-import)
W: 24, 0: Unused import Exp3 from wildcard import (unused-wildcard-import)
W: 24, 0: Unused import MusicalChair from wildcard import (unused-wildcard-import)
W: 24, 0: Unused import Uniform from wildcard import (unused-wildcard-import)
W: 24, 0: Unused import EpsilonDecreasing from wildcard import (unused-wildcard-import)
W: 24, 0: Unused import EpsilonGreedy from wildcard import (unused-wildcard-import)
W: 26, 0: Unused klucbGauss imported from Policies.kullback (unused-import)
W: 26, 0: Unused klucbExp imported from Policies.kullback (unused-import)
W: 26, 0: Unused klucbBern imported from Policies.kullback (unused-import)
R: 98, 0: Redefinition of DECREASE_RATE type from float to str (redefined-variable-type)
R: 76, 0: Redefinition of TEMPERATURE type from float to int (redefined-variable-type)
************* Module Arms.Arm
C: 21, 4: Missing method docstring (missing-docstring)
C: 24, 4: Missing method docstring (missing-docstring)
C: 27, 4: Missing method docstring (missing-docstring)
C: 33, 4: Missing method docstring (missing-docstring)
C: 37, 4: Missing method docstring (missing-docstring)
************* Module configuration_multiplayers
W: 29, 0: XXX do better for these imports (fixme)
W: 35, 0: XXX Should be >= 10000 to be interesting "asymptotically" (fixme)
W: 48, 0: XXX to disable this optimization (fixme)
W: 51, 0: XXX Should be >= 10 to be statistically trustworthy (fixme)
W: 52, 0: XXX To profile the code, turn down parallel computing (fixme)
W: 58, 0: XXX To profile the code, turn down parallel computing (fixme)
W: 60, 0: XXX do not let this = False # To profile the code, turn down parallel computing (fixme)
W: 82, 0: XXX More than the number of arms !! (fixme)
W: 87, 0: XXX this is the best one (fixme)
W:102, 0: XXX This dictionary configures the experiments (fixme)
W:146, 0: XXX To only test the orthogonalization (collision avoidance) protocol (fixme)
W:171, 0: XXX compute optimal values for d (MEGA's parameter) (fixme)
W:194, 0: FIXME this Selfish[AdBandits] and Selfish[BayesUCB] work crazily well... ? (fixme)
W:199, 0: XXX each player needs to now the number of players (fixme)
W:204, 0: FIXME try it ! (fixme)
W:207, 0: XXX they need a perfect knowledge on the arms, OF COURSE this is not physically plausible at all (fixme)
W:219, 0: FIXME how to chose the 5 parameters ?? (fixme)
W:223, 0: FIXME how to chose the 2 parameters p0 and alpha_p0 ? (fixme)
W:225, 0: TODO try this one! (fixme)
W:226, 0: TODO try this one! (fixme)
W:227, 0: TODO try this one! (fixme)
W:228, 0: TODO try this one! (fixme)
W:229, 0: TODO try this one! (fixme)
W:230, 0: TODO try this one! (fixme)
W:231, 0: TODO try this one! (fixme)
W:242, 0: TODO the EvaluatorMultiPlayers should regenerate the list of players in every repetitions, to have at the end results on the average behavior of these randomized multi-players policies (fixme)
C:206, 0: Wrong continued indentation (add 11 spaces).
# --- DONE Using multi-player Oracle policy
^ | (bad-continuation)
C:211, 0: Wrong continued indentation (add 11 spaces).
# --- DONE Using single-player Musical Chair policy
^ | (bad-continuation)
C:218, 0: Wrong continued indentation (add 11 spaces).
# --- DONE Using single-player MEGA policy
^ | (bad-continuation)
C:222, 0: Wrong continued indentation (add 11 spaces).
# --- FIXME Using single-player ALOHA policy
^ | (bad-continuation)
C:233, 0: Wrong continued indentation (add 11 spaces).
# --- DONE Using single-player rhoRand policy
^ | (bad-continuation)
W: 27, 0: Wildcard import Policies (wildcard-import)
W: 28, 0: Wildcard import PoliciesMultiPlayers (wildcard-import)
W: 31, 0: Wildcard import Environment.CollisionModels (wildcard-import)
W: 20, 0: Unused Exponential imported from Arms.Exponential (unused-import)
W: 21, 0: Unused Gaussian imported from Arms.Gaussian (unused-import)
W: 22, 0: Unused Poisson imported from Arms.Poisson (unused-import)
W: 25, 0: Unused MAB imported from Environment.MAB (unused-import)
W: 27, 0: Unused import UCBopt from wildcard import (unused-wildcard-import)
W: 27, 0: Unused import klUCBHPlus from wildcard import (unused-wildcard-import)
W: 27, 0: Unused import TakeFixedArm from wildcard import (unused-wildcard-import)
W: 27, 0: Unused import Aggr from wildcard import (unused-wildcard-import)
W: 27, 0: Unused import klUCB from wildcard import (unused-wildcard-import)
W: 27, 0: Unused import UCBV from wildcard import (unused-wildcard-import)
W: 27, 0: Unused import SoftmaxDecreasing from wildcard import (unused-wildcard-import)
W: 27, 0: Unused import Exp3Decreasing from wildcard import (unused-wildcard-import)
W: 27, 0: Unused import klUCBPlus from wildcard import (unused-wildcard-import)
W: 27, 0: Unused import UCBrandomInit from wildcard import (unused-wildcard-import)
W: 27, 0: Unused import UCBwrong from wildcard import (unused-wildcard-import)
W: 27, 0: Unused import AdBandits from wildcard import (unused-wildcard-import)
W: 27, 0: Unused import UCBalpha from wildcard import (unused-wildcard-import)
W: 27, 0: Unused import SoftMix from wildcard import (unused-wildcard-import)
W: 27, 0: Unused import BayesUCB from wildcard import (unused-wildcard-import)
W: 27, 0: Unused import EpsilonExpDecreasing from wildcard import (unused-wildcard-import)
W: 27, 0: Unused import TakeRandomFixedArm from wildcard import (unused-wildcard-import)
W: 27, 0: Unused import EpsilonGreedy from wildcard import (unused-wildcard-import)
W: 27, 0: Unused import MOSS from wildcard import (unused-wildcard-import)
W: 27, 0: Unused import Exp3 from wildcard import (unused-wildcard-import)
W: 27, 0: Unused import Softmax from wildcard import (unused-wildcard-import)
W: 27, 0: Unused import UCBVtuned from wildcard import (unused-wildcard-import)
W: 27, 0: Unused import EpsilonDecreasingMEGA from wildcard import (unused-wildcard-import)
W: 27, 0: Unused import MEGA from wildcard import (unused-wildcard-import)
W: 27, 0: Unused import EpsilonDecreasing from wildcard import (unused-wildcard-import)
W: 27, 0: Unused import UCBplus from wildcard import (unused-wildcard-import)
W: 27, 0: Unused import UCB from wildcard import (unused-wildcard-import)
W: 27, 0: Unused import UniformOnSome from wildcard import (unused-wildcard-import)
W: 27, 0: Unused import Exp3SoftMix from wildcard import (unused-wildcard-import)
W: 27, 0: Unused import EpsilonFirst from wildcard import (unused-wildcard-import)
W: 27, 0: Unused import Exp3WithHorizon from wildcard import (unused-wildcard-import)
W: 27, 0: Unused import SoftmaxWithHorizon from wildcard import (unused-wildcard-import)
W: 27, 0: Unused import MusicalChair from wildcard import (unused-wildcard-import)
W: 27, 0: Unused import Uniform from wildcard import (unused-wildcard-import)
W: 28, 0: Unused import Selfish from wildcard import (unused-wildcard-import)
W: 28, 0: Unused import ALOHA from wildcard import (unused-wildcard-import)
W: 28, 0: Unused import Scenario1 from wildcard import (unused-wildcard-import)
W: 28, 0: Unused import rhoRand from wildcard import (unused-wildcard-import)
W: 28, 0: Unused import tnext_beta from wildcard import (unused-wildcard-import)
W: 28, 0: Unused import tnext_log from wildcard import (unused-wildcard-import)
W: 28, 0: Unused import OracleNotFair from wildcard import (unused-wildcard-import)
W: 28, 0: Unused import OracleFair from wildcard import (unused-wildcard-import)
W: 28, 0: Unused import CentralizedCycling from wildcard import (unused-wildcard-import)
W: 28, 0: Unused import CentralizedFixed from wildcard import (unused-wildcard-import)
W: 31, 0: Unused import collision_models from wildcard import (unused-wildcard-import)
W: 31, 0: Unused import random_distances from wildcard import (unused-wildcard-import)
W: 31, 0: Unused import handleCollision_or_getZeroReward from wildcard import (unused-wildcard-import)
W: 31, 0: Unused import closerUserGetsReward from wildcard import (unused-wildcard-import)
W: 31, 0: Unused import defaultCollisionModel from wildcard import (unused-wildcard-import)
W: 31, 0: Unused import lru_cache from wildcard import (unused-wildcard-import)
C: 31, 0: Imports from package Environment are not grouped (ungrouped-imports)
************* Module main_multiplayers
W: 28, 0: XXX dont keep it like this (fixme)
W: 87, 0: XXX To plot without saving (fixme)
W: 96, 0: XXX To plot without saving (fixme)
W:105, 0: XXX To plot without saving (fixme)
W:114, 0: XXX To plot without saving (fixme)
W:123, 0: XXX To plot without saving (fixme)
W:132, 0: XXX To plot without saving (fixme)
W:141, 0: XXX To plot without saving (fixme)
W:150, 0: XXX To plot without saving (fixme)
W:159, 0: XXX To plot without saving (fixme)
W:168, 0: XXX To plot without saving (fixme)
W:170, 0: XXX Also plotting the cumulative number of pulls of all arms (fixme)
W:177, 0: XXX To plot without saving (fixme)
W:186, 0: XXX To plot without saving (fixme)
W:195, 0: XXX To plot without saving (fixme)
W:206, 0: XXX To plot without saving (fixme)
************* Module main
W: 32, 0: XXX dont keep it like this when experimenting (fixme)
W: 74, 0: FIXME finish this! (fixme)
W: 96, 0: TODO use numpy.savez_compressed instead ? https://docs.scipy.org/doc/numpy/reference/generated/numpy.savez_compressed.html#numpy.savez_compressed (fixme)
W:116, 0: XXX To save the figure (fixme)
W:118, 0: XXX To save the figure (fixme)
W:128, 0: XXX To save the figure (fixme)
W:137, 0: XXX To save the figure (fixme)
W:150, 0: XXX To save the figure (fixme)
C: 58, 0: Wrong continued indentation (remove 1 space).
)
|^ (bad-continuation)
C: 17, 0: standard import "import pickle" comes before "import matplotlib.pyplot as plt" (wrong-import-order)
************* Module Arms.kullback
W:376, 0: XXX where does this value come from? (fixme)
W:381, 0: XXX where does this value come from? (fixme)
W: 21,11: Redefining name 'x' from outer scope (line 492) (redefined-outer-name)
W: 45,14: Redefining name 'x' from outer scope (line 492) (redefined-outer-name)
W: 71,10: Redefining name 'x' from outer scope (line 492) (redefined-outer-name)
W:104,12: Redefining name 'x' from outer scope (line 492) (redefined-outer-name)
W:137,13: Redefining name 'x' from outer scope (line 492) (redefined-outer-name)
W:176,12: Redefining name 'x' from outer scope (line 492) (redefined-outer-name)
W:223,10: Redefining name 'x' from outer scope (line 492) (redefined-outer-name)
W:223,13: Redefining name 'd' from outer scope (line 494) (redefined-outer-name)
W:245,17: Redefining name 'd' from outer scope (line 494) (redefined-outer-name)
W:245,14: Redefining name 'x' from outer scope (line 492) (redefined-outer-name)
W:279,18: Redefining name 'd' from outer scope (line 494) (redefined-outer-name)
W:279,15: Redefining name 'x' from outer scope (line 492) (redefined-outer-name)
W:279,30: Unused argument 'precision' (unused-argument)
W:314,20: Redefining name 'd' from outer scope (line 494) (redefined-outer-name)
W:314,17: Redefining name 'x' from outer scope (line 492) (redefined-outer-name)
W:347,16: Redefining name 'd' from outer scope (line 494) (redefined-outer-name)
W:347,13: Redefining name 'x' from outer scope (line 492) (redefined-outer-name)
W:390,16: Redefining name 'klMax' from outer scope (line 478) (redefined-outer-name)
W:390,10: Redefining name 'p' from outer scope (line 474) (redefined-outer-name)
W:390,13: Redefining name 'V' from outer scope (line 476) (redefined-outer-name)
W:426,17: Redefining name 'klMax' from outer scope (line 478) (redefined-outer-name)
W:426,11: Redefining name 'p' from outer scope (line 474) (redefined-outer-name)
W:426,14: Redefining name 'V' from outer scope (line 476) (redefined-outer-name)
W:456, 4: String statement has no effect (pointless-string-statement)
************* Module Environment.CollisionModels
W: 29, 0: XXX could it help? (fixme)
W: 47, 0: XXX Should player.getReward() be called with a reward = 0 when there is collisions (to change the internals memory of the player) ? (fixme)
W: 48, 0: XXX Strong assumption on the model (fixme)
W:116, 0: XXX Using a cache to not regenerate a random vector of distances. Siooooux! (fixme)
W:117, 0: XXX size is NOT bounded... bad! (fixme)
W:151, 0: XXX very low probability, if the distances are randomly chosen (fixme)
C: 33, 8: Missing function docstring (missing-docstring)
W: 31,31: Unused argument 'typed' (unused-argument)
W: 31,18: Unused argument 'maxsize' (unused-argument)
W: 76,59: Unused argument 'collisions' (unused-argument)
C: 98, 4: Consider using enumerate instead of iterating with range and len (consider-using-enumerate)
C:137, 4: Consider using enumerate instead of iterating with range and len (consider-using-enumerate)
************* Module Environment.MAB
W: 98, 0: FIXME it is weird to have a lowerbound = 0 if nbPlayers = nbArms (fixme)
C: 83, 0: Wrong continued indentation (add 20 spaces).
)
^ | (bad-continuation)
C: 62, 4: Missing method docstring (missing-docstring)
C:123, 4: Missing method docstring (missing-docstring)
************* Module Environment.ResultMultiPlayers
C: 23, 4: Missing method docstring (missing-docstring)
R: 10, 0: Too few public methods (1/2) (too-few-public-methods)
************* Module Environment.Result
W: 27, 0: FIXME find a way to store the result while learning? (fixme)
C: 21, 4: Missing method docstring (missing-docstring)
R: 10, 0: Too few public methods (1/2) (too-few-public-methods)
************* Module Environment.plotsettings
W: 16, 0: FIXME revert to ↑ after having generating the figures for the paper (fixme)
W: 22, 0: XXX turn on XKCD-like style ?! cf. http://matplotlib.org/xkcd/ for more details (fixme)
C: 29, 0: Wrong continued indentation (remove 1 space).
)
|^ (bad-continuation)
W: 61, 4: No exception type(s) specified (bare-except)
W: 65, 8: No exception type(s) specified (bare-except)
W: 69,12: No exception type(s) specified (bare-except)
W: 73,16: No exception type(s) specified (bare-except)
************* Module Policies.AdBandits
W: 81, 0: XXX What happens here if two arms has the same index, being the max? (fixme)
W: 41, 4: Arguments number differs from overridden 'getReward' method (arguments-differ)
C: 48, 4: Missing method docstring (missing-docstring)
************* Module Policies.BasePolicy
W: 41, 0: XXX we could check here if the reward is outside the bounds (fixme)
C: 33, 4: Missing method docstring (missing-docstring)
C: 38, 4: Missing method docstring (missing-docstring)
C: 50, 4: Missing method docstring (missing-docstring)
C: 55, 4: Missing method docstring (missing-docstring)
C: 61, 4: Missing method docstring (missing-docstring)
C: 67, 4: Missing method docstring (missing-docstring)
************* Module Policies.BayesianIndexPolicy
W: 25, 4: Arguments number differs from overridden 'getReward' method (arguments-differ)
************* Module Policies.Aggr
W: 21, 0: XXX Better (fixme)
W: 25, 0: XXX Better (fixme)
W: 49, 0: FIXME talk about this (fixme)
W: 50, 0: XXX If we use the Exp4 update rule, it's better to be unbiased (fixme)
W: 51, 0: XXX If we use my update rule, it seems to be better to be "biased" (fixme)
W:141, 0: FIXME try this trick of receiving a loss instead of a reward ? (fixme)
W:160, 0: FIXME experiment dynamic resetting of proba, put this as a parameter (fixme)
W:172, 0: XXX No: in fact, we need to vector self.choices to update the self.trusts probabilities! (fixme)
C: 42, 0: Wrong continued indentation (remove 1 space).
):
|^ (bad-continuation)
R: 32, 0: Too many instance attributes (15/7) (too-many-instance-attributes)
W: 39,17: Redefining name 'update_all_children' from outer scope (line 16) (redefined-outer-name)
W: 39,58: Redefining name 'update_like_exp4' from outer scope (line 24) (redefined-outer-name)
W: 40,17: Redefining name 'unbiased' from outer scope (line 20) (redefined-outer-name)
W: 37, 4: __init__ method from base class 'BasePolicy' is not called (super-init-not-called)
W:124, 4: Arguments number differs from overridden 'getReward' method (arguments-differ)
************* Module Policies.EpsilonDecreasingMEGA
C: 18, 0: Missing function docstring (missing-docstring)
************* Module Policies.EpsilonGreedy
W: 48, 0: XXX What happens here if two arms has the same index, being the max? (fixme)
C: 28, 4: Missing method docstring (missing-docstring)
E: 51,51: Instance of 'EpsilonGreedy' has no 'index' member (no-member)
E: 72,51: Instance of 'EpsilonGreedy' has no 'index' member (no-member)
************* Module Policies.Beta
W: 35, 4: __init__ method from base class 'Posterior' is not called (super-init-not-called)
W: 43, 4: Arguments number differs from overridden 'reset' method (arguments-differ)
************* Module Policies.Exp3
W: 56, 0: XXX Call to BasePolicy (fixme)
W: 70, 0: TODO? random permutation instead of deterministic order! (fixme)
C: 46, 4: Missing method docstring (missing-docstring)
C: 50, 4: Missing method docstring (missing-docstring)
W: 55, 4: Arguments number differs from overridden 'getReward' method (arguments-differ)
W:119, 4: Arguments number differs from overridden 'gamma' method (arguments-differ)
************* Module Policies.IndexPolicy
W: 64, 0: XXX What happens here if two arms has the same index, being the max? (fixme)
W: 78, 0: FIXME if no arms are tagged as available, what to do ? choose an arm at random, or call choice() as if available == 'all' (fixme)
************* Module Policies.__init__
W: 59, 0: XXX fix it before importing it (fixme)
W: 71, 0: FIXME write it! Can be just a subclass of MusicalChair (fixme)
************* Module Policies.klUCBHPlus
W: 34, 0: XXX We could adapt tolerance to the value of self.t (fixme)
E: 11, 0: No name 'klucbBern' in module 'Policies.kullback' (no-name-in-module)
************* Module Policies.klUCBPlus
W: 23, 0: XXX We could adapt tolerance to the value of self.t (fixme)
************* Module Environment.Evaluator
W: 99, 0: XXX Use this method to speed up computation (fixme)
W:150, 0: FIXME this BestArmPulls is wrong in case of dynamic change of arm configurations (fixme)
W:188, 0: XXX make it look smaller, for the plots (fixme)
W:194, 0: XXX make it look larger, for the plots (fixme)
W:209, 0: XXX put back 0 if needed (fixme)
W:217, 0: XXX prevent /0 (fixme)
W:224, 0: XXX plt.fill_between http://matplotlib.org/users/recipes.html#fill-between-and-alpha instead of plt.errorbar (fixme)
W:234, 0: XXX Not smart, if maxmu = 0.1 we don't see anything (fixme)
W:267, 0: XXX put back 0 if needed (fixme)
W:309, 0: XXX Try to give a unique seed to random & numpy.random for each call of this function (fixme)
W:321, 0: XXX Experimental support for random events: shuffling or inverting the list of arms, at these time steps (fixme)
W:330, 0: XXX do this quicker! (fixme)
W:338, 0: XXX inefficient and does not work yet (fixme)
W:343, 0: XXX Experimental : shuffle the arms at the middle of the simulation (fixme)
W:345, 0: XXX improve this: it is slow to test 'in <a list>', faster to compute a 't % ...' (fixme)
W:348, 0: XXX Experimental : invert the order of the arms at the middle of the simulation (fixme)
W:350, 0: XXX improve this: it is slow to test 'in <a list>', faster to compute a 't % ...' (fixme)
C:128, 0: Wrong continued indentation (add 25 spaces).
random_shuffle=self.random_shuffle, random_invert=self.random_invert, nb_random_events=self.nb_random_events,
^ | (bad-continuation)
C:129, 0: Wrong continued indentation (add 25 spaces).
delta_t_save=self.delta_t_save, allrewards=allrewards, seed=seeds[i]
^ | (bad-continuation)
C:130, 0: Wrong continued indentation (add 28 spaces).
)
^ | (bad-continuation)
C:137, 0: Wrong continued indentation (add 13 spaces).
random_shuffle=self.random_shuffle, random_invert=self.random_invert, nb_random_events=self.nb_random_events,
^ | (bad-continuation)
C:138, 0: Wrong continued indentation (add 13 spaces).
delta_t_save=self.delta_t_save, allrewards=allrewards
^ | (bad-continuation)
C:139, 0: Wrong continued indentation (add 12 spaces).
)
^ | (bad-continuation)
C:203, 0: Wrong continued indentation (remove 1 space).
):
|^ (bad-continuation)
R: 32, 0: Too many instance attributes (22/7) (too-many-instance-attributes)
C:106, 4: Missing method docstring (missing-docstring)
C:111, 4: Missing method docstring (missing-docstring)
C:155, 4: Missing method docstring (missing-docstring)
C:158, 4: Missing method docstring (missing-docstring)
C:162, 4: Missing method docstring (missing-docstring)
C:165, 4: Missing method docstring (missing-docstring)
C:168, 4: Missing method docstring (missing-docstring)
C:172, 4: Missing method docstring (missing-docstring)
C:175, 4: Missing method docstring (missing-docstring)
C:178, 4: Missing method docstring (missing-docstring)
C:201, 4: Missing method docstring (missing-docstring)
R:201, 4: Too many local variables (20/15) (too-many-locals)
C:262, 4: Missing method docstring (missing-docstring)
C:284, 4: Missing method docstring (missing-docstring)
W:104, 8: Attribute 'allrewards' defined outside __init__ (attribute-defined-outside-init)
W:307,17: Redefining name 'random_shuffle' from outer scope (line 27) (redefined-outer-name)
W:307,48: Redefining name 'random_invert' from outer scope (line 28) (redefined-outer-name)
W:307,77: Redefining name 'nb_random_events' from outer scope (line 29) (redefined-outer-name)
C:306, 0: Missing function docstring (missing-docstring)
************* Module Policies.KLempUCB
W: 11, 0: XXX Not detected as in the kullback.py file ? (fixme)
W: 46, 0: FIXME this does not work apparently... (fixme)
E: 11, 0: No name 'maxEV' in module 'Policies.kullback' (no-name-in-module)
W: 41, 4: Arguments number differs from overridden 'getReward' method (arguments-differ)
C: 47, 4: Missing method docstring (missing-docstring)
R: 47, 4: Method could be a function (no-self-use)
************* Module Policies.klUCB
W: 32, 0: XXX We could adapt tolerance to the value of self.t (fixme)
E: 12, 0: No name 'klucbBern' in module 'Policies.kullback' (no-name-in-module)
************* Module Policies.MEGA
W: 52, 0: FIXME we should not require two parameters, as they are just used in the formula for epsilon_t (fixme)
W: 87, 0: XXX Chose a random arm (fixme)
W: 97, 0: TODO should be uniformly chosen if more than one arm has the highest index, but that's unlikely (fixme)
R: 28, 0: Too many instance attributes (10/7) (too-many-instance-attributes)
W:102, 4: Arguments number differs from overridden 'getReward' method (arguments-differ)
C:136, 4: Missing method docstring (missing-docstring)
************* Module PoliciesMultiPlayers.BaseCentralizedPolicy
C: 17, 4: Missing method docstring (missing-docstring)
C: 20, 4: Missing method docstring (missing-docstring)
C: 23, 4: Missing method docstring (missing-docstring)
************* Module PoliciesMultiPlayers.BaseMPPolicy
R: 11, 0: Too few public methods (0/2) (too-few-public-methods)
************* Module Environment.EvaluatorMultiPlayers
W:126, 0: FIXME having this list of results consumes too much RAM ! (fixme)
W:136, 0: FIXME there is probably a bug in this computation (fixme)
W:173, 0: FIXME it depend on the collision model ! (fixme)
W:192, 0: XXX put back 0 if needed (fixme)
W:199, 0: XXX Should be smarter (fixme)
W:204, 0: TODO add std (fixme)
W:216, 0: FIXME I should compute a certain measure of "fairness", from these personal rewards (fixme)
W:218, 0: XXX It was weird to use a std/var (fixme)
W:219, 0: TODO try it! (fixme)
W:239, 0: XXX prevent /0 (fixme)
W:250, 0: TODO add std (fixme)
W:279, 0: XXX put back 0 if needed (fixme)
W:285, 0: XXX Should be smarter (fixme)
W:326, 0: XXX put back 0 if needed (fixme)
W:357, 0: TODO should only plot with markers (fixme)
W:369, 0: TODO I should plot the evolution of the occupation ratio of each channel, as a function of time (fixme)
W:384, 0: XXX To normalized the count? (fixme)
W:404, 0: XXX no, we should not count just the fact that there were collisions, but instead count all collisions (fixme)
W:427, 0: TODO do an histogram instead of this piechart (fixme)
W:429, 0: XXX if this is not enough, do the histogram/bar plot manually, and add labels as texts (fixme)
W:470, 0: XXX Try to give a unique seed to random & numpy.random for each call of this function (fixme)
W:499, 0: FIXME here, the environment should apply a random permutation to each player, in order for the non-modified UCB-like algorithms to work fine in case of collisions (their initial exploration phase is non-random hence leading to only collisions in the first steps, and ruining the performance) (fixme)
W:500, 0: FIXME try it! (fixme)
W:504, 0: XXX It is here that the player may receive a reward, if there is no collisions (fixme)
W:418, 0: Anomalous backslash in string: '\%'. String constant might be missing an r prefix. (anomalous-backslash-in-string)
R: 31, 0: Too many instance attributes (20/7) (too-many-instance-attributes)
C:100, 4: Missing method docstring (missing-docstring)
C:105, 4: Missing method docstring (missing-docstring)
C:141, 4: Missing method docstring (missing-docstring)
C:144, 4: Missing method docstring (missing-docstring)
C:147, 4: Missing method docstring (missing-docstring)
C:150, 4: Missing method docstring (missing-docstring)
C:154, 4: Missing method docstring (missing-docstring)
C:157, 4: Missing method docstring (missing-docstring)
C:160, 4: Missing method docstring (missing-docstring)
C:168, 4: Missing method docstring (missing-docstring)
C:186, 4: Missing method docstring (missing-docstring)
R:186, 4: Too many local variables (17/15) (too-many-locals)
C:235, 4: Missing method docstring (missing-docstring)
C:272, 4: Missing method docstring (missing-docstring)
R:272, 4: Too many local variables (16/15) (too-many-locals)
C:302, 4: Missing method docstring (missing-docstring)
C:320, 4: Missing method docstring (missing-docstring)
R:320, 4: Too many local variables (16/15) (too-many-locals)
C:348, 4: Missing method docstring (missing-docstring)
C:373, 4: Missing method docstring (missing-docstring)
C:397, 4: Missing method docstring (missing-docstring)
C:438, 4: Missing method docstring (missing-docstring)
C:455, 4: Missing method docstring (missing-docstring)
R: 31, 0: Too many public methods (21/20) (too-many-public-methods)
C:468, 0: Missing function docstring (missing-docstring)
R:468, 0: Too many local variables (16/15) (too-many-locals)
************* Module PoliciesMultiPlayers.ALOHA
W: 79, 0: XXX Call ChildPointer method (fixme)
W: 97, 0: XXX Call ChildPointer method (fixme)
W:107, 0: XXX should we give a 0 reward ? (fixme)
W:123, 0: FIXME check the algorithm (fixme)
W:126, 0: XXX Call ChildPointer method (fixme)
C: 27, 4: Missing function docstring (missing-docstring)
C: 39, 4: Missing function docstring (missing-docstring)
R: 46, 0: Too many instance attributes (10/7) (too-many-instance-attributes)
W: 54,17: Unused argument 'lower' (unused-argument)
W: 54,27: Unused argument 'amplitude' (unused-argument)
W: 76,92: Access to a protected member _players of a client class (protected-access)
C: 85, 4: Missing method docstring (missing-docstring)
W:138, 4: __init__ method from base class 'BaseMPPolicy' is not called (super-init-not-called)
C:181, 4: Missing method docstring (missing-docstring)
C:184, 4: Missing method docstring (missing-docstring)
C:187, 4: Missing method docstring (missing-docstring)
R:134, 0: Too few public methods (0/2) (too-few-public-methods)
************* Module PoliciesMultiPlayers.CentralizedFixed
W: 73, 0: XXX this "trash" arm with max number of collision will not change: that can be very good (if it is the worse!) or very bad (if it is the best!) (fixme)
W:104, 0: FIXME It should re-generate the affectations every time a game is started! (fixme)
W: 25, 4: __init__ method from base class 'BaseCentralizedPolicy' is not called (super-init-not-called)
W: 25,41: Unused argument 'lower' (unused-argument)
W: 25,51: Unused argument 'amplitude' (unused-argument)
W: 46, 4: __init__ method from base class 'BaseMPPolicy' is not called (super-init-not-called)
C:103, 4: Missing method docstring (missing-docstring)
C:107, 4: Missing method docstring (missing-docstring)
C:110, 4: Missing method docstring (missing-docstring)
R: 42, 0: Too few public methods (0/2) (too-few-public-methods)
************* Module PoliciesMultiPlayers.CentralizedCycling
W: 27, 0: XXX It could be a random permutation, instead of a cycling (fixme)
W: 76, 0: XXX this "trash" arm with max number of collision will cycle: that's the best we can do! (fixme)
W:104, 0: FIXME It should re-generate the affectations every time a game is started! (fixme)
W: 24, 4: __init__ method from base class 'BaseCentralizedPolicy' is not called (super-init-not-called)
W: 48, 4: __init__ method from base class 'BaseMPPolicy' is not called (super-init-not-called)
W: 48,42: Unused argument 'lower' (unused-argument)
W: 48,52: Unused argument 'amplitude' (unused-argument)
C:103, 4: Missing method docstring (missing-docstring)
C:107, 4: Missing method docstring (missing-docstring)
C:110, 4: Missing method docstring (missing-docstring)
R: 44, 0: Too few public methods (0/2) (too-few-public-methods)
************* Module PoliciesMultiPlayers.CentralizedMultiplePlay
W: 32, 4: __init__ method from base class 'BaseMPPolicy' is not called (super-init-not-called)
C: 61, 4: Missing method docstring (missing-docstring)
C: 66, 4: Missing method docstring (missing-docstring)
C: 72, 4: Missing method docstring (missing-docstring)
C: 79, 4: Missing method docstring (missing-docstring)
W: 79,35: Unused argument 'playerId' (unused-argument)
W: 79,45: Unused argument 'arm' (unused-argument)
R: 79, 4: Method could be a function (no-self-use)
R: 28, 0: Too few public methods (0/2) (too-few-public-methods)
W: 74,12: Attribute 'choices' defined outside __init__ (attribute-defined-outside-init)
************* Module PoliciesMultiPlayers.ChildPointer
W: 18,51: Access to a protected member _players of a client class (protected-access)
W: 21,27: Access to a protected member _players of a client class (protected-access)
W: 25,15: Access to a protected member _startGame_one of a client class (protected-access)
W: 29,15: Access to a protected member _getReward_one of a client class (protected-access)
W: 33,15: Access to a protected member _choice_one of a client class (protected-access)
W: 37,15: Access to a protected member _choiceWithRank_one of a client class (protected-access)
W: 41,15: Access to a protected member _choiceFromSubSet_one of a client class (protected-access)
W: 45,15: Access to a protected member _choiceMultiple_one of a client class (protected-access)
W: 49,15: Access to a protected member _handleCollision_one of a client class (protected-access)
************* Module PoliciesMultiPlayers.__init__
W: 36, 0: FIXME implement it (fixme)
************* Module Policies.kullback
W:376, 0: XXX where does this value come from? (fixme)
W:381, 0: XXX where does this value come from? (fixme)
W: 21,11: Redefining name 'x' from outer scope (line 492) (redefined-outer-name)
W: 45,14: Redefining name 'x' from outer scope (line 492) (redefined-outer-name)
W: 71,10: Redefining name 'x' from outer scope (line 492) (redefined-outer-name)
W:104,12: Redefining name 'x' from outer scope (line 492) (redefined-outer-name)
W:137,13: Redefining name 'x' from outer scope (line 492) (redefined-outer-name)
W:176,12: Redefining name 'x' from outer scope (line 492) (redefined-outer-name)
W:223,10: Redefining name 'x' from outer scope (line 492) (redefined-outer-name)
W:223,13: Redefining name 'd' from outer scope (line 494) (redefined-outer-name)
W:245,17: Redefining name 'd' from outer scope (line 494) (redefined-outer-name)
W:245,14: Redefining name 'x' from outer scope (line 492) (redefined-outer-name)
W:279,18: Redefining name 'd' from outer scope (line 494) (redefined-outer-name)
W:279,15: Redefining name 'x' from outer scope (line 492) (redefined-outer-name)
W:279,30: Unused argument 'precision' (unused-argument)
W:314,20: Redefining name 'd' from outer scope (line 494) (redefined-outer-name)
W:314,17: Redefining name 'x' from outer scope (line 492) (redefined-outer-name)
W:347,16: Redefining name 'd' from outer scope (line 494) (redefined-outer-name)
W:347,13: Redefining name 'x' from outer scope (line 492) (redefined-outer-name)
W:390,16: Redefining name 'klMax' from outer scope (line 478) (redefined-outer-name)
W:390,10: Redefining name 'p' from outer scope (line 474) (redefined-outer-name)
W:390,13: Redefining name 'V' from outer scope (line 476) (redefined-outer-name)
W:426,17: Redefining name 'klMax' from outer scope (line 478) (redefined-outer-name)
W:426,11: Redefining name 'p' from outer scope (line 474) (redefined-outer-name)
W:426,14: Redefining name 'V' from outer scope (line 476) (redefined-outer-name)
W:456, 4: String statement has no effect (pointless-string-statement)
************* Module PoliciesMultiPlayers.OracleFair
W: 85, 0: FIXME improve this, when there is more player than arms, this is not optimal (fixme)
W: 86, 0: FIXME indeed, all the collisions will first be in worseArm, but they all cycle! That's bad (fixme)
W: 87, 0: XXX this "trash" arm with max number of collision will cycle: that's the best we can do! (fixme)
W:116, 0: FIXME It should re-generate the affectations every time a game is started! (fixme)
W: 26, 4: __init__ method from base class 'BaseCentralizedPolicy' is not called (super-init-not-called)
W: 37, 8: Unreachable code (unreachable)
W: 54, 4: __init__ method from base class 'BaseMPPolicy' is not called (super-init-not-called)
W: 54,43: Unused argument 'lower' (unused-argument)
W: 54,53: Unused argument 'amplitude' (unused-argument)
C:115, 4: Missing method docstring (missing-docstring)
C:119, 4: Missing method docstring (missing-docstring)
C:122, 4: Missing method docstring (missing-docstring)
R: 50, 0: Too few public methods (0/2) (too-few-public-methods)
************* Module PoliciesMultiPlayers.OracleNotFair
W: 77, 0: XXX this "trash" arm with max number of collision will not change, but it's the worse arm, so we are optimal! (fixme)
W:105, 0: FIXME It should re-generate the affectations every time a game is started! (fixme)
W: 27, 4: __init__ method from base class 'BaseCentralizedPolicy' is not called (super-init-not-called)
W: 48, 4: __init__ method from base class 'BaseMPPolicy' is not called (super-init-not-called)
W: 48,43: Unused argument 'lower' (unused-argument)
W: 48,53: Unused argument 'amplitude' (unused-argument)
C:104, 4: Missing method docstring (missing-docstring)
C:108, 4: Missing method docstring (missing-docstring)
C:111, 4: Missing method docstring (missing-docstring)
R: 44, 0: Too few public methods (0/2) (too-few-public-methods)
************* Module PoliciesMultiPlayers.rhoRand
W: 50, 0: FIXME here we could do another randomization step: (fixme)
W: 52, 0: TODO to try! (fixme)
W: 37,73: Access to a protected member _players of a client class (protected-access)
W: 39,64: Access to a protected member _players of a client class (protected-access)
W: 64, 4: __init__ method from base class 'BaseMPPolicy' is not called (super-init-not-called)
C: 92, 4: Missing method docstring (missing-docstring)
C: 95, 4: Missing method docstring (missing-docstring)
C: 98, 4: Missing method docstring (missing-docstring)
R: 60, 0: Too few public methods (0/2) (too-few-public-methods)
************* Module Policies.Posterior
C: 14, 4: Missing method docstring (missing-docstring)
C: 17, 4: Missing method docstring (missing-docstring)
C: 20, 4: Missing method docstring (missing-docstring)
C: 23, 4: Missing method docstring (missing-docstring)
C: 26, 4: Missing method docstring (missing-docstring)
C: 29, 4: Missing method docstring (missing-docstring)
************* Module PoliciesMultiPlayers.Selfish
W: 68, 0: FIXME Strong assumption on the model (fixme)
W: 22, 4: __init__ method from base class 'BaseMPPolicy' is not called (super-init-not-called)
C: 51, 4: Missing method docstring (missing-docstring)
C: 54, 4: Missing method docstring (missing-docstring)
C: 57, 4: Missing method docstring (missing-docstring)
C: 60, 4: Missing method docstring (missing-docstring)
R: 15, 0: Too few public methods (0/2) (too-few-public-methods)
************* Module PoliciesMultiPlayers.Scenario1
W: 78, 0: XXX to be general for nbArms > 2 setting (fixme)
W: 40, 4: __init__ method from base class 'BaseMPPolicy' is not called (super-init-not-called)
W: 40,42: Unused argument 'lower' (unused-argument)
W: 40,52: Unused argument 'amplitude' (unused-argument)
C: 65, 4: Missing method docstring (missing-docstring)
W: 65,29: Unused argument 'playerId' (unused-argument)
C: 68, 4: Missing method docstring (missing-docstring)
C: 71, 4: Missing method docstring (missing-docstring)
R: 34, 0: Too few public methods (0/2) (too-few-public-methods)
************* Module Policies.TakeFixedArm
W: 17, 4: __init__ method from base class 'BasePolicy' is not called (super-init-not-called)
W: 27, 4: Arguments number differs from overridden 'getReward' method (arguments-differ)
************* Module Policies.TakeRandomFixedArm
W: 17, 4: __init__ method from base class 'TakeFixedArm' is not called (super-init-not-called)
************* Module Policies.Softmax
W: 36, 0: XXX trying to randomize the order of the initial visit to each arm; as this determinism breaks its habitility to play efficiently in multi-players games (fixme)
W: 37, 0: XXX do even more randomized, take a random permutation of the arm (fixme)
W: 54, 0: XXX we don't need this, the BasePolicy.getReward does it already (fixme)
W: 57, 0: FIXME we should divide by the proba p_t of selecting actions, not by the trusts ! (fixme)
W: 69, 0: TODO? random permutation instead of deterministic order! (fixme)
W: 90, 0: XXX wrong size if n > 1 (fixme)
C: 81, 0: Unnecessary parens after 'if' keyword (superfluous-parens)
C: 49, 4: Missing method docstring (missing-docstring)
C: 53, 4: Missing method docstring (missing-docstring)
W:122, 4: Arguments number differs from overridden 'temperature' method (arguments-differ)
************* Module Policies.MusicalChair
W: 99, 0: XXX it will then be of size nbPlayers! (fixme)
W:143, 0: TODO remove this (fixme)
R: 64, 0: Too many instance attributes (9/7) (too-many-instance-attributes)
W:146, 4: Arguments number differs from overridden 'getReward' method (arguments-differ)
C:165, 4: Missing method docstring (missing-docstring)
W:183,30: Unused argument 'arm' (unused-argument)
************* Module Policies.UCBrandomInit
W: 20, 0: XXX trying to randomize the order of the initial visit to each arm; as this determinism breaks its habitility to play efficiently in multi-players games (fixme)
W: 21, 0: XXX do even more randomized, take a random permutation of the arm (fixme)
************* Module Policies.UCBwrong
W: 31, 0: XXX Volontary typo, wrong mean estimate (fixme)
************* Module Policies.UCBV
W: 28, 4: Arguments number differs from overridden 'getReward' method (arguments-differ)
************* Module Policies.UniformOnSome
W: 17, 4: __init__ method from base class 'Uniform' is not called (super-init-not-called)
************* Module Policies.Uniform
W: 16, 4: __init__ method from base class 'BasePolicy' is not called (super-init-not-called)
W: 22, 4: Arguments number differs from overridden 'getReward' method (arguments-differ)
Report
======
3152 statements analysed.
Statistics by type
------------------
+---------+-------+-----------+-----------+------------+---------+
|type |number |old number |difference |%documented |%badname |
+=========+=======+===========+===========+============+=========+
|module |73 |72 |+1.00 |98.63 |5.48 |
+---------+-------+-----------+-----------+------------+---------+
|class |72 |70 |+2.00 |100.00 |8.33 |
+---------+-------+-----------+-----------+------------+---------+
|method |365 |0 |+365.00 |73.70 |23.56 |
+---------+-------+-----------+-----------+------------+---------+
|function |52 |0 |+52.00 |88.46 |61.54 |
+---------+-------+-----------+-----------+------------+---------+
External dependencies
---------------------
::
Arms (configuration_multiplayers)
\-Arm (Arms.Uniform)
\-Bernoulli (configuration_multiplayers)
\-Exponential (configuration_multiplayers)
\-Gaussian (configuration_multiplayers)
\-Poisson (configuration_multiplayers)
\-kullback (Arms.Poisson)
Environment
\-CollisionModels (Environment.EvaluatorMultiPlayers)
\-Evaluator (main)
\-EvaluatorMultiPlayers (main_multiplayers)
\-MAB (Environment.EvaluatorMultiPlayers)
\-Result (Environment.Evaluator)
\-ResultMultiPlayers (Environment.EvaluatorMultiPlayers)
\-plotsettings (Environment.EvaluatorMultiPlayers)
Policies
\-BasePolicy (Policies.Uniform)
\-BayesianIndexPolicy (Policies.Thompson)
\-Beta (Policies.BayesianIndexPolicy)
\-EpsilonGreedy (Policies.EpsilonFirst)
\-IndexPolicy (Policies.UCBwrong)
\-Posterior (Policies.Beta)
\-TakeFixedArm (Policies.TakeRandomFixedArm)
\-UCB (Policies.UCBV)
\-UCBV (Policies.UCBVtuned)
\-klUCB (Policies.klUCBPlus)
\-kullback (Policies.klUCB)
PoliciesMultiPlayers
\-ALOHA (configuration_multiplayers)
\-BaseCentralizedPolicy (PoliciesMultiPlayers.OracleNotFair)
\-BaseMPPolicy (PoliciesMultiPlayers.Scenario1)
\-ChildPointer (PoliciesMultiPlayers.Scenario1)
\-Scenario1 (configuration)
configuration
\-configuration (main)
configuration_multiplayers
\-configuration (main_multiplayers)
joblib (Environment.EvaluatorMultiPlayers)
matplotlib
\-pyplot (Environment.EvaluatorMultiPlayers)
numpy (Policies.UCBV)
\-random (Policies.TakeRandomFixedArm)
recommonmark
\-parser (conf)
scipy
\-special (Policies.Beta)
\-stats (Arms.Poisson)
seaborn (Environment.plotsettings)
sphinx_rtd_theme (conf)
Raw metrics
-----------
+----------+-------+------+---------+-----------+
|type |number |% |previous |difference |
+==========+=======+======+=========+===========+
|code |3538 |48.79 |3498 |+40.00 |
+----------+-------+------+---------+-----------+
|docstring |1484 |20.47 |1463 |+21.00 |
+----------+-------+------+---------+-----------+
|comment |1171 |16.15 |1152 |+19.00 |
+----------+-------+------+---------+-----------+
|empty |1058 |14.59 |1040 |+18.00 |
+----------+-------+------+---------+-----------+
Duplication
-----------
+-------------------------+------+---------+-----------+
| |now |previous |difference |
+=========================+======+=========+===========+
|nb duplicated lines |0 |0 |= |
+-------------------------+------+---------+-----------+
|percent duplicated lines |0.000 |0.000 |= |
+-------------------------+------+---------+-----------+
Messages by category
--------------------
+-----------+-------+---------+-----------+
|type |number |previous |difference |
+===========+=======+=========+===========+
|convention |125 |0 |+125.00 |
+-----------+-------+---------+-----------+
|refactor |29 |0 |+29.00 |
+-----------+-------+---------+-----------+
|warning |408 |0 |+408.00 |
+-----------+-------+---------+-----------+
|error |8 |0 |+8.00 |
+-----------+-------+---------+-----------+
% errors / warnings by module
-----------------------------
+---------------------------------------------+------+--------+---------+-----------+
|module |error |warning |refactor |convention |
+=============================================+======+========+=========+===========+
|configuration |37.50 |15.44 |6.90 |0.80 |
+---------------------------------------------+------+--------+---------+-----------+
|Policies.EpsilonGreedy |25.00 |0.25 |0.00 |0.80 |
+---------------------------------------------+------+--------+---------+-----------+
|Policies.KLempUCB |12.50 |0.74 |3.45 |0.80 |
+---------------------------------------------+------+--------+---------+-----------+
|Policies.klUCBHPlus |12.50 |0.25 |0.00 |0.00 |
+---------------------------------------------+------+--------+---------+-----------+
|Policies.klUCB |12.50 |0.25 |0.00 |0.00 |
+---------------------------------------------+------+--------+---------+-----------+
|configuration_multiplayers |0.00 |20.34 |0.00 |4.80 |
+---------------------------------------------+------+--------+---------+-----------+
|Policies.kullback |0.00 |6.37 |0.00 |0.00 |
+---------------------------------------------+------+--------+---------+-----------+
|Arms.kullback |0.00 |6.37 |0.00 |0.00 |
+---------------------------------------------+------+--------+---------+-----------+
|Environment.EvaluatorMultiPlayers |0.00 |6.13 |20.69 |16.80 |
+---------------------------------------------+------+--------+---------+-----------+
|Environment.Evaluator |0.00 |5.15 |6.90 |16.80 |
+---------------------------------------------+------+--------+---------+-----------+
|main_multiplayers |0.00 |3.92 |0.00 |0.00 |
+---------------------------------------------+------+--------+---------+-----------+
|Policies.Aggr |0.00 |3.19 |3.45 |0.80 |
+---------------------------------------------+------+--------+---------+-----------+
|PoliciesMultiPlayers.ALOHA |0.00 |2.21 |6.90 |4.80 |
+---------------------------------------------+------+--------+---------+-----------+
|PoliciesMultiPlayers.OracleFair |0.00 |2.21 |3.45 |2.40 |
+---------------------------------------------+------+--------+---------+-----------+
|Environment.CollisionModels |0.00 |2.21 |0.00 |2.40 |
+---------------------------------------------+------+--------+---------+-----------+
|PoliciesMultiPlayers.ChildPointer |0.00 |2.21 |0.00 |0.00 |
+---------------------------------------------+------+--------+---------+-----------+
|main |0.00 |1.96 |0.00 |1.60 |
+---------------------------------------------+------+--------+---------+-----------+
|PoliciesMultiPlayers.CentralizedCycling |0.00 |1.72 |3.45 |2.40 |
+---------------------------------------------+------+--------+---------+-----------+
|Policies.Softmax |0.00 |1.72 |0.00 |2.40 |
+---------------------------------------------+------+--------+---------+-----------+
|PoliciesMultiPlayers.OracleNotFair |0.00 |1.47 |3.45 |2.40 |
+---------------------------------------------+------+--------+---------+-----------+
|PoliciesMultiPlayers.CentralizedFixed |0.00 |1.47 |3.45 |2.40 |
+---------------------------------------------+------+--------+---------+-----------+
|Environment.plotsettings |0.00 |1.47 |0.00 |0.80 |
+---------------------------------------------+------+--------+---------+-----------+
|PoliciesMultiPlayers.rhoRand |0.00 |1.23 |3.45 |2.40 |
+---------------------------------------------+------+--------+---------+-----------+
|PoliciesMultiPlayers.Scenario1 |0.00 |1.23 |3.45 |2.40 |
+---------------------------------------------+------+--------+---------+-----------+
|PoliciesMultiPlayers.CentralizedMultiplePlay |0.00 |0.98 |6.90 |3.20 |
+---------------------------------------------+------+--------+---------+-----------+
|Policies.MusicalChair |0.00 |0.98 |3.45 |0.80 |
+---------------------------------------------+------+--------+---------+-----------+
|Policies.MEGA |0.00 |0.98 |3.45 |0.80 |
+---------------------------------------------+------+--------+---------+-----------+
|Policies.Exp3 |0.00 |0.98 |0.00 |1.60 |
+---------------------------------------------+------+--------+---------+-----------+
|PoliciesMultiPlayers.Selfish |0.00 |0.49 |3.45 |3.20 |
+---------------------------------------------+------+--------+---------+-----------+
|Policies.AdBandits |0.00 |0.49 |0.00 |0.80 |
+---------------------------------------------+------+--------+---------+-----------+
|Policies.__init__ |0.00 |0.49 |0.00 |0.00 |
+---------------------------------------------+------+--------+---------+-----------+
|Policies.Uniform |0.00 |0.49 |0.00 |0.00 |
+---------------------------------------------+------+--------+---------+-----------+
|Policies.UCBrandomInit |0.00 |0.49 |0.00 |0.00 |
+---------------------------------------------+------+--------+---------+-----------+
|Policies.TakeFixedArm |0.00 |0.49 |0.00 |0.00 |
+---------------------------------------------+------+--------+---------+-----------+
|Policies.IndexPolicy |0.00 |0.49 |0.00 |0.00 |
+---------------------------------------------+------+--------+---------+-----------+
|Policies.Beta |0.00 |0.49 |0.00 |0.00 |
+---------------------------------------------+------+--------+---------+-----------+
|conf |0.00 |0.25 |3.45 |0.80 |
+---------------------------------------------+------+--------+---------+-----------+
|Environment.Result |0.00 |0.25 |3.45 |0.80 |
+---------------------------------------------+------+--------+---------+-----------+
|Policies.BasePolicy |0.00 |0.25 |0.00 |4.80 |
+---------------------------------------------+------+--------+---------+-----------+
|Environment.MAB |0.00 |0.25 |0.00 |2.40 |
+---------------------------------------------+------+--------+---------+-----------+
|PoliciesMultiPlayers.__init__ |0.00 |0.25 |0.00 |0.00 |
+---------------------------------------------+------+--------+---------+-----------+
|Policies.klUCBPlus |0.00 |0.25 |0.00 |0.00 |
+---------------------------------------------+------+--------+---------+-----------+
|Policies.UniformOnSome |0.00 |0.25 |0.00 |0.00 |
+---------------------------------------------+------+--------+---------+-----------+
|Policies.UCBwrong |0.00 |0.25 |0.00 |0.00 |
+---------------------------------------------+------+--------+---------+-----------+
|Policies.UCBV |0.00 |0.25 |0.00 |0.00 |
+---------------------------------------------+------+--------+---------+-----------+
|Policies.TakeRandomFixedArm |0.00 |0.25 |0.00 |0.00 |
+---------------------------------------------+------+--------+---------+-----------+
|Policies.BayesianIndexPolicy |0.00 |0.25 |0.00 |0.00 |
+---------------------------------------------+------+--------+---------+-----------+
|Environment.ResultMultiPlayers |0.00 |0.00 |3.45 |0.80 |
+---------------------------------------------+------+--------+---------+-----------+
|PoliciesMultiPlayers.BaseMPPolicy |0.00 |0.00 |3.45 |0.00 |
+---------------------------------------------+------+--------+---------+-----------+
|Policies.Posterior |0.00 |0.00 |0.00 |4.80 |
+---------------------------------------------+------+--------+---------+-----------+
|Arms.Arm |0.00 |0.00 |0.00 |4.00 |
+---------------------------------------------+------+--------+---------+-----------+
|PoliciesMultiPlayers.BaseCentralizedPolicy |0.00 |0.00 |0.00 |2.40 |
+---------------------------------------------+------+--------+---------+-----------+
|Policies.EpsilonDecreasingMEGA |0.00 |0.00 |0.00 |0.80 |
+---------------------------------------------+------+--------+---------+-----------+
Messages
--------
+-------------------------------+------------+
|message id |occurrences |
+===============================+============+
|fixme |189 |
+-------------------------------+------------+
|missing-docstring |103 |
+-------------------------------+------------+
|unused-wildcard-import |78 |
+-------------------------------+------------+
|redefined-outer-name |50 |
+-------------------------------+------------+
|unused-argument |21 |
+-------------------------------+------------+
|super-init-not-called |19 |
+-------------------------------+------------+
|bad-continuation |16 |
+-------------------------------+------------+
|arguments-differ |13 |
+-------------------------------+------------+
|too-few-public-methods |12 |
+-------------------------------+------------+
|protected-access |12 |
+-------------------------------+------------+
|unused-import |11 |
+-------------------------------+------------+
|too-many-instance-attributes |6 |
+-------------------------------+------------+
|no-name-in-module |6 |
+-------------------------------+------------+
|too-many-locals |5 |
+-------------------------------+------------+
|wildcard-import |4 |
+-------------------------------+------------+
|bare-except |4 |
+-------------------------------+------------+
|redefined-variable-type |3 |
+-------------------------------+------------+
|pointless-string-statement |2 |
+-------------------------------+------------+
|no-self-use |2 |
+-------------------------------+------------+
|no-member |2 |
+-------------------------------+------------+
|consider-using-enumerate |2 |
+-------------------------------+------------+
|attribute-defined-outside-init |2 |
+-------------------------------+------------+
|wrong-import-position |1 |
+-------------------------------+------------+
|wrong-import-order |1 |
+-------------------------------+------------+
|unreachable |1 |
+-------------------------------+------------+
|ungrouped-imports |1 |
+-------------------------------+------------+
|too-many-public-methods |1 |
+-------------------------------+------------+
|superfluous-parens |1 |
+-------------------------------+------------+
|redefined-builtin |1 |
+-------------------------------+------------+
|anomalous-backslash-in-string |1 |
+-------------------------------+------------+
Global evaluation
-----------------
Your code has been rated at 8.09/10 (previous run: 10.00/10, -1.91)