liangfu commented on issue #4293: [VTA] Bug fix for padded load with large 
inputs
URL: https://github.com/apache/incubator-tvm/pull/4293#issuecomment-554241928
 
 
   @vegaluisjose Yes, current Chisel implement passes all the unit-tests.
   
   @tmoreau89 I did not test this with HLS version yet. If you want, I think I 
can test this with PYNQ hardware next week.
   
   On the other hand, I think some recent changes (either in the runtime or the 
compiler) has made this broken. I mean even without this PR, unit test with 
`fsim` could even fail in some cases (not always). I can reproduce the failure 
case by fixing random seed. All my successful tests are based on eariler 
version of runtime & compiler. This PR only focus on the hardware improvements.

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

Reply via email to