Closed thaarres closed 2 years ago
Could @thesps, @vloncar, @nicologhielmetti , @jmduarte have a look and let me know if anything is missing/should be corrected?
Thanks Thea, this looks great. BTW, since we merged line-buffer implementation the way algorithm works (by default) has been changed so we should use the new animation from that PR.
Ah great! Thanks for the link and for having a look @vloncar! I’ll change that section asap.
On 29 Jun 2021, at 16:46, vloncar @.**@.>> wrote:
Thanks Thea, this looks great. BTW, since we merged line-bufferhttps://github.com/fastmachinelearning/hls4ml/pull/332 implementation the way algorithm works (by default) has been changed so we should use the new animation from that PR.
— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHubhttps://github.com/fastmachinelearning/hls4ml-tutorial/pull/16#issuecomment-870665595, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ABKLVQCIRHYTSABIFHFKLMLTVHMETANCNFSM47P62JGQ.
@thesps anything missng to merge this?
This was included in #23, so now merged. Closing this one!
Tutorial on CNNs in hls4ml, based on https://iopscience.iop.org/article/10.1088/2632-2153/ac0ea1