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

Graph with 4900 nodes takes a long time to render #140

Closed
justinchuby opened this issue Sep 3, 2024 · 4 comments
Closed

Graph with 4900 nodes takes a long time to render #140

justinchuby opened this issue Sep 3, 2024 · 4 comments
Assignees
@pkgoogle
Labels
status:awaiting user response This label needs to be added to stale issues and PRs. status:more data needed This label needs to be added to stale issues and PRs.

Comments

@justinchuby
Copy link
Contributor

justinchuby commented Sep 3, 2024 •
edited
Loading

It's been like this for many minutes and hasn't rendered yet (netron renders it easily):
image

Graph:
big-lama.onnx.json
@pkgoogle
Copy link
Contributor

pkgoogle commented Sep 3, 2024

Hi @justinchuby, is this intended to be used w/ your ONNX adapter (i.e. should I remove the .json and use your ONNX adapter) or with the raw adapter?

@pkgoogle pkgoogle added ONNX adapter For issues where the root is mostly occurring with the ONNX adapter status:awaiting user response This label needs to be added to stale issues and PRs. status:more data needed This label needs to be added to stale issues and PRs. labels Sep 3, 2024
@justinchuby
Copy link
Contributor Author

It is the json file I saved from the UI so the raw adapter should work? If needed I can provide the ONNX model.

@pkgoogle
Copy link
Contributor

pkgoogle commented Sep 4, 2024

Hi @justinchuby, I am able to load/render properly on both Mac and Linux (Debian) with the default adapter and the .json file as is. Can you ensure your package is updated? or let me know what version of the ai-edge-model-explorer package you are using?

@pkgoogle pkgoogle removed the ONNX adapter For issues where the root is mostly occurring with the ONNX adapter label Sep 4, 2024
@justinchuby
Copy link
Contributor Author

It does work now for some reason! Maybe my computer was slow that day. I was using 0.10. Feel free to mark this as done, thanks for taking a look.

@pkgoogle pkgoogle closed this as completed Sep 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@pkgoogle pkgoogle

Labels
status:awaiting user response This label needs to be added to stale issues and PRs. status:more data needed This label needs to be added to stale issues and PRs.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@justinchuby @pkgoogle