Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

MLPerf Tiny developments #503

Open
wants to merge 117 commits into
base: main
Choose a base branch
from

Conversation

jmduarte
Copy link
Member

@jmduarte jmduarte commented Mar 11, 2022

Not to be merged but simply to open discussion about which developments we should cherry-pick and rebase to hls4ml main branch. Based on a quick look, I can imagine a few separate PRs:

nghielme and others added 30 commits March 8, 2021 10:49
…vcd file. Next step -> add the parser, find the max for each fifo and set the new fifo depth
* fix conv1d io_parallel resource

* replace tabs with spaces in modified code

* Add test for Conv1D example model

Co-authored-by: Sioni Summers <[email protected]>
@thesps
Copy link
Contributor

thesps commented Mar 15, 2022

Hi @jmduarte, all the updates sounds good to me. I know @nicologhielmetti has begun the work to port the FIFO depth optimization to an optimizer pass on top of the current hls4ml framework.

For the "ReLU merge optimizer" do you have any numbers for the savings (latency or resources) that result?

@anmeza
Copy link

anmeza commented Apr 1, 2022

@thesps Here is a table showing the resource utilization (estimated via csynth) before and after applying the "ReLU merge optimization" to an image classification model used for the tinyMLPerf submission.

Screenshot (204)

@oliviaweng oliviaweng mentioned this pull request Jun 29, 2022
7 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

10 participants