search

tum-ei-eda / mlonmcu

Tool for the deployment and analysis of TinyML applications on TFLM and MicroTVM backends
Apache License 2.0
29 stars 13 forks source link

Less pip install issue51 #80

Closed zhouxinyu0723 closed 1 year ago

PhilippvK commented 1 year ago

Please check the linter issues. Most of the stuff can be resolved using black -l 120 mlonmcu

PhilippvK commented 1 year ago

mlonmcu/read_write_file_lock/.nfs80fc025d08848a620000031f should probably be removed

zhouxinyu0723 commented 1 year ago

should I fix the black and .gitignore and make a new commit and a new PR?

PhilippvK commented 1 year ago

No, just add new commits to the pr branch

PhilippvK commented 1 year ago

@zhouxinyu0723 Thank you. Black issues are resolved now, while anything which is not Flake8 (a bit stricter linter) compliant can be found here and needs to be fixed manually:

https://github.com/tum-ei-eda/mlonmcu/pull/80/checks?check_run_id=9005475013

PhilippvK commented 1 year ago

Look very good @zhouxinyu0723. Thank you!