search

StanfordAHA / garnet

Next generation CGRA generator
BSD 3-Clause "New" or "Revised" License
99 stars 11 forks source link

Remove 3000 lint errors #1056

Closed steveri closed 3 months ago

steveri commented 3 months ago

Scrubbed three mflowgen/*/construct.py files to remove lint errors. This brings overall garnet repo lint PEP8 style error count down from 9978 to 6337:

BEFORE: https://buildkite.com/stanford-aha/garnet/builds/4981 ...........................Found 9978 E errors

AFTER: https://buildkite.com/stanford-aha/garnet/builds/4992 ...........................Found 6337 E errors

steveri commented 3 months ago

Looks good. This happened because these construct.py files were written based on the sample construct.py files from the mflowgen repo, which don't conform to PEP8.

@alexcarsello thanks very much for the review! I didn't know if you would care, so I didn't include you formally on the list, although I knew I probably should have...