forked from prob-ml/bliss
-
Notifications
You must be signed in to change notification settings - Fork 0
/
.flake8
131 lines (90 loc) · 2.87 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
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
[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,
# so we can import survey objects with acronyms
N817,
# 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,RST203,RST210
# we don't care about security/cryptography
S311,
# upper-case constants are fine in Enums
WPS115
# there are a number of places where a float zero value makes sense
WPS358
# what's wrong a multiline try block?
WPS229,
# lambda expressions are more concise and very readable
E731,
# we should consider fixing some of these issues
W503,WPS202,WPS213,WPS214,WPS231,WPS236,WPS336,WPS338,WPS440,WPS441,WPS442,WPS602