search

cornell-zhang / heterocl

HeteroCL: A Multi-Paradigm Programming Infrastructure for Software-Defined Heterogeneous Computing
https://cornell-zhang.github.io/heterocl/
Apache License 2.0
326 stars 92 forks source link

[CI] Add attrs and specify black, pylint version #499

Closed zzzDavid closed 1 year ago

zzzDavid commented 1 year ago

attrs is added to the dependency to solve this issue:

heterocl/devices.py:9:0: E0401: Unable to import 'attr' (import-error)
heterocl/tools.py:7:0: E0401: Unable to import 'attr' (import-error)

I think we should also specify a version of black and pylint to use. Pylint and black may have different check results with different versions. The undergrad team has encountered this issue a few time, I think it's easier to just fix a version on CI and local test copy.