Skip to content

Commit

Permalink
Merge pull request cleverhans-lab#776 from goodfeli/W1201
Browse files Browse the repository at this point in the history
Closes cleverhans-lab#613 (W1201)
  • Loading branch information
npapernot authored Oct 12, 2018
2 parents 518f4eb + 18dde6a commit 3118b3e
Showing 1 changed file with 9 additions and 0 deletions.
9 changes: 9 additions & 0 deletions .pylintrc
Original file line number Diff line number Diff line change
Expand Up @@ -80,6 +80,15 @@ msg-template='{path}:{line}: [{msg_id}({symbol}), {obj}] {msg}'
# R0902: too many attributes
# C0200: consider using enumerate
# W1201: logging not lazy
# Disabled because it forbids things like
# logging.info('#' * pad + msg + '#' * pad)
# Doesn't seem useful to refactor this to
# padded_msg = '#' * pad + msg + '#' * pad
# logging.info(padded_msg)
# or to refactor it to
# logging.info('%s%s%s', pad, msg, pad)
# The speedup of avoiding a string concatenation is presumably negligible
# compared to most of the ML operations we do to generate log messages.
# W1202: logging format
# C0302: too many lines in module
# W0511: forbids TODOs
Expand Down

0 comments on commit 3118b3e

Please sign in to comment.