You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@tvm.apache.org by GitBox <gi...@apache.org> on 2021/09/03 04:42:38 UTC

[GitHub] [tvm-vta] tmoreau89 commented on pull request #32: VTA Chisel Wide memory interface.

tmoreau89 commented on pull request #32:
URL: https://github.com/apache/tvm-vta/pull/32#issuecomment-912250501


   Thanks @vegaluisjose for reviewing the PR and ensuring that the CI tests go green, much appreciated. While going through the PR there were a few things that I'd like to address before we merge:
   
   - Still quite a few comments that need to be cleaned up
   - The comments could use a good spell check pass, many typos in some files - perhaps it could be automated via the preferred IDE
   - Overall I think this is not blocking the merge, but we could extend the scala linter to address consistent comment formatting etc.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@tvm.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org