Jovan Mitrevski

Results 133 comments of Jovan Mitrevski

The CNN fixes are actually in #610. You are right that it should not be here. I am not sure if it's best to try to get that in first...

I fixed up #610 so maybe it's easier to merge this after that PR (and I fix the remaining issues).

The onnx converter has gone through many changes since 0.5.0, but having said that, the newest version still fails on this model, though in a different way. We are actively...

The qkeras pytest failure is a known problem that exists in the main branch even without this change.

The last push was not a complete fix. I'll send a message again when I think it's ready.

I still need to test vsynth=True and also that I didn't mess up the FIFO optimizations.

Actually, looking in the main branch as a starting point, I see ``` 'VivadoSynthReport': {'LUT': '0', 'FF': '0', 'BRAM_18K': '0', 'URAM': '0', 'DSP48E': '0'} ``` when I do part1 of...

This seems to break the VivadoAccelerator backend.

I believe this is ready for merging.

For the nondefault project name handling, it may be good to rebase with the current main branch. I think those things have been solved, though of course, there are no...