forked from prob-ml/bliss
-
Notifications
You must be signed in to change notification settings - Fork 0
/
.flake8
113 lines (78 loc) · 2.52 KB
/
.flake8
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
[flake8]
exclude = .git
max-line-length = 100
inline-quotes = double
docstring-convention=google
ignore =
# documentation not yet needed
D100,D101,D102,D103,D104,D105,D106,D107,
# high Jones complexity isn't always so bad
WPS221,
# removing all magic numbers seems like overkill
WPS432,
# assertions are useful for development code
S101,
# requiring trailing commas seems too picky
C812,
# nested imports can be fine
WPS433,
# number of local variables is already checked by pylint
WPS210,
# a redundant subscript can be more readible that defining a temp value
WPS349,
# not sure what is wrong with comparing an array to a float
WPS459,
# only makes sense once we're using proper docstrings
D205, D400, DAR201, DAR101,
# useless else has its moments
WPS503,
# no need to explicitly inherit from object
WPS306,
# for list comprehensions one-character variable names are useful
WPS111,
# some of these bad variable names are set by other packages
WPS110,
# too many extra lines
WPS414,
# seems to work fine
WPS465,
# already enforced by pylint
WPS211,
# not too many assert statements
WPS218,
# reusing string constants is less verbose that defining a variable
WPS226,
# positional arguments to formatting strings are redundant
P101,
# I am not worried about security issues associate with pickle
S301, S403,
# black disagrees, actually inserting space before some colons
E203,
# f-strings are OK, though perhaps using .format is better
WPS305,
# some of these "overused" expressions are quite short
WPS204,
# these distinction btw public and private class attrs isn't that meaningful here
WPS230,
# false positive; this cannot be done with "yield from"
WPS526,
# not helpful
WPS201,
# not that complex really
WPS237,
# unfortunately pytorch convention is to import "functional" as F
N812,
# it has to be empty
WPS411,
# code is too verbose if you don't import submodules
WPS458,
# negated condition can be more clear
WPS504,
# implicit string concatenation seems pretty clear
WPS326,
# the keyword "del" is useful to us
WPS420,
# darglint validates our doc strings
RST201,RST301,D202
# we should consider fixing some of these issues
E731,W503,WPS202,WPS213,WPS214,WPS231,WPS232,WPS236,WPS336,WPS338,WPS345,WPS347,WPS358,WPS421,WPS440,WPS441,WPS442,WPS602