Fix Flake8 update and pin version

awslabs / benchmark-ai

Anubis (formerly known as Benchmark AI), measures the goodness of machine learning workloads

Apache License 2.0

16 stars 6 forks source link

Fix Flake8 update and pin version #1028

Closed Chancebair closed 4 years ago

Chancebair commented 4 years ago

Issue #, if available: https://github.com/awslabs/benchmark-ai/issues/1026

Description of changes: Flake8 updated and found some stuff, I pinned the version and fixed the lint failures

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

gavinmbell commented 4 years ago

Hi... What errors are we going to be suppressing?

--ignore=E902 seems to suppress IO/Errors (https://pep8.readthedocs.io/en/latest/intro.html)

--cosmetic-- I see removing all these "f" format print statements as clean up - since there is no formatting for them to work on. I would advocate for this being two diff PRs.