tum-ei-eda / mlonmcu

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

run tvmc with temp working dir; fix model group lookup #63

Closed rafzi closed 2 years ago

PhilippvK commented 2 years ago

@rafzi can you shortly explain what was wrong about the model group handling? I thought I fixed it last week.

rafzi commented 2 years ago

The index i for the list replacement didn't really make sense as that was referring to the index of the group list, not the model list.

For the linting, I found out that I have to add --experimental-string-processing to black to make it format that line.

PhilippvK commented 2 years ago

@rafzi thanks. Make sense to me. I will try out the model group handling tomorrow and try to add a unit test which would have catched this issue.