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.

Sign up for GitHub

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

Jump to bottom

Improve the performance of the ice40_import_routing_from_icebox.py tool #196

Open
mithro opened this issue Jul 15, 2018 · 2 comments
Open

Improve the performance of the ice40_import_routing_from_icebox.py tool #196

mithro opened this issue Jul 15, 2018 · 2 comments
Labels
arch-ice40 Issue related to the iCE40 architecture description. lang-python Issue uses (or requires) Python language. lang-xml Issues uses (or requires) XML language. type-utils Issues is related to the scripts inside the repo.

Comments

@mithro
Copy link
Contributor

mithro commented Jul 15, 2018

Currently the ice40_import_routing_from_icebox.py tool was written to make debugging easy but not to be fast.

It would be nice if the script was much faster! Get the Python profiler out and see what is taking most of the time. Then fix that issue!
@mithro mithro added lang-xml Issues uses (or requires) XML language. lang-python Issue uses (or requires) Python language. arch-ice40 Issue related to the iCE40 architecture description. type-utils Issues is related to the scripts inside the repo. labels Jul 15, 2018
@litghost
Copy link
Contributor

I've profiled it already, and ~60% of the time is spent adding XML nodes for the edges in the graph.

@litghost litghost self-assigned this Aug 22, 2018
@litghost litghost removed their assignment Jan 31, 2019
@elms elms added this to Needs triage in Code Health Mar 26, 2019
@elms elms mentioned this issue Mar 26, 2019
Closed
@elms elms moved this from Needs triage to High priority in Code Health Apr 3, 2019
@j340m3
Copy link

j340m3 commented Dec 7, 2020 •
edited

The link is dead. Does this file exist anymore? If so, could you update the reference? Thank you! @mithro @litghost
EDIT: NVM, found it: here

@kgugala kgugala removed this from High priority in Code Health Sep 3, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
arch-ice40 Issue related to the iCE40 architecture description. lang-python Issue uses (or requires) Python language. lang-xml Issues uses (or requires) XML language. type-utils Issues is related to the scripts inside the repo.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@mithro @litghost @j340m3