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

Refresh architecture diagram with ONNX and LiteRT inputs. #19494

Merged
merged 1 commit into from
Dec 16, 2024

Conversation

ScottTodd
Copy link
Member

Progress on #18174.

I suggest the rich diff "swipe" view to see what changed. Updates in text form:

  • Added ONNX to input frameworks in the top left and swapped "TOSA" for "LiteRT".
  • Reordered the input frameworks (roughly ordered by momentum and grouped TF with LiteRT, but could also sort alphabetically).
  • Exporting somehow changed the font to from [Virgil, Cascadia, Assistant] sourced from e.g. https://excalidraw.com/Cascadia.woff2 to just Cascadia, sourced from an inlined base64-encoded blob (so +1.87KB file size). I don't mind the changes.
  • Removed extremely small "Stacked Column" text from the "Profiling Feedback" chart on the right that looked like an artifact of the diagram generation.

@ScottTodd ScottTodd added documentation ✏️ Improvements or additions to documentation integrations/onnx ONNX integration work labels Dec 16, 2024
@ScottTodd ScottTodd requested review from benvanik and marbre December 16, 2024 21:22
@benvanik
Copy link
Collaborator

nice!

@ScottTodd ScottTodd merged commit 8e86bcf into iree-org:main Dec 16, 2024
27 checks passed
@ScottTodd ScottTodd deleted the docs-diagram-refresh branch December 16, 2024 21:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation ✏️ Improvements or additions to documentation integrations/onnx ONNX integration work
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants