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

[nnx] use HashableMapping instead of FrozenDict #4376

Merged

copybara-service merged 1 commit into main from nnx-benchmark

Nov 13, 2024

Collaborator

cgarciae commented Nov 12, 2024 •

edited

Loading

What does this PR do?

Changes NodeDef.index_mapping from FrozenDict to HashableMapping for performance.

IvyZX approved these changes

View reviewed changes


          [nnx] use HashableMapping instead of FrozenDict

347211b

cgarciae force-pushed the nnx-benchmark branch from f37a3da to 347211b Compare

November 13, 2024 00:44

cgarciae added the pull ready label

copybara-service bot merged commit 480a196 into main

19 checks passed

copybara-service bot deleted the nnx-benchmark branch

November 13, 2024 02:29

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels