nncf icon indicating copy to clipboard operation
nncf copied to clipboard

[Torch FX] Map Namespace Names to Metatype

Open anzr299 opened this issue 1 year ago • 3 comments

Changes

Created a new dictionary in operator metatypes registry to maintain mapping of namespace and function name to metatype.

Reason for changes

For a more accurate retrieval of operation metatype.

Tests

original graph tests and metatypes were used to check the correctness of the change.

anzr299 avatar Feb 03 '25 11:02 anzr299

@anzr299, please fix the pre-commit: KeyError: 'PTInputNoopMetatype is already registered in input_noop_metatypes'

daniil-lyakhov avatar Feb 05 '25 09:02 daniil-lyakhov

https://github.com/openvinotoolkit/nncf/pull/3258#discussion_r1952621554

cc' @AlexanderDokuchaev

alexsu52 avatar Feb 13 '25 04:02 alexsu52

A test case have caught the issue with an embedding model here https://github.com/ynimmaga/executorch/pull/26 @anzr299, please prioritize this fix

daniil-lyakhov avatar Feb 26 '25 12:02 daniil-lyakhov