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

[TorchFX] Model transformer tests #2920

Merged
merged 4 commits into from
Oct 16, 2024
Merged

[TorchFX] Model transformer tests #2920

merged 4 commits into from
Oct 16, 2024

Conversation

daniil-lyakhov
Copy link
Collaborator

@daniil-lyakhov daniil-lyakhov commented Aug 27, 2024 •
edited
Loading

Changes

Model transformation tests are presented

Reason for changes

To cover TorchFX model transformations by tests

Related tickets

#2775

Tests

  • test_model_insertion_transformation
  • test_constant_update_transformation
  • test_constant_update_transformation_no_constant
  • TestQDQInsertion
  • test_node_removal_transformation

@github-actions github-actions bot added NNCF PT Pull requests that updates NNCF PyTorch experimental NNCF OpenVINO Pull requests that updates NNCF OpenVINO NNCF ONNX Pull requests that updates NNCF ONNX NNCF PTQ Pull requests that updates NNCF PTQ labels Aug 27, 2024
@daniil-lyakhov daniil-lyakhov force-pushed the dl/fx/model_transformer_tests_actual branch 2 times, most recently from fdf6f72 to 1d42c0d Compare August 27, 2024 17:21
This was referenced Aug 27, 2024
Closed
Closed
@daniil-lyakhov daniil-lyakhov force-pushed the dl/fx/model_transformer_tests_actual branch from 1d42c0d to f9fd4a2 Compare August 27, 2024 17:45
@daniil-lyakhov daniil-lyakhov force-pushed the dl/fx/model_transformer_tests_actual branch from f9fd4a2 to 44746bc Compare September 19, 2024 12:08
@github-actions github-actions bot added experimental NNCF PTQ Pull requests that updates NNCF PTQ and removed experimental NNCF OpenVINO Pull requests that updates NNCF OpenVINO NNCF ONNX Pull requests that updates NNCF ONNX NNCF PTQ Pull requests that updates NNCF PTQ labels Sep 19, 2024
@daniil-lyakhov daniil-lyakhov mentioned this pull request Sep 20, 2024
Merged
@daniil-lyakhov daniil-lyakhov force-pushed the dl/fx/model_transformer_tests_actual branch 2 times, most recently from c33e6ca to 31b6f26 Compare September 20, 2024 13:04
@daniil-lyakhov daniil-lyakhov force-pushed the dl/fx/model_transformer_tests_actual branch 3 times, most recently from c7e7b3b to 83d9ea4 Compare September 30, 2024 14:39
@daniil-lyakhov daniil-lyakhov marked this pull request as ready for review September 30, 2024 14:41
@daniil-lyakhov daniil-lyakhov requested a review from a team as a code owner September 30, 2024 14:41
@daniil-lyakhov
Copy link
Collaborator Author

@AlexanderDokuchaev, please take a look

@daniil-lyakhov daniil-lyakhov changed the title [WIP][TorchFX] Model transformer tests [TorchFX] Model transformer tests Oct 14, 2024
AlexanderDokuchaev
AlexanderDokuchaev requested changes Oct 14, 2024
View reviewed changes
nncf/quantization/algorithms/smooth_quant/torch_fx_backend.py Show resolved Hide resolved
tests/torch/fx/test_model_transformer.py Outdated Show resolved Hide resolved
tests/torch/fx/test_model_transformer.py Outdated Show resolved Hide resolved
tests/torch/fx/test_model_transformer.py Outdated Show resolved Hide resolved
tests/torch/fx/test_model_transformer.py Outdated Show resolved Hide resolved
nncf/experimental/torch/fx/transformations.py Show resolved Hide resolved
@daniil-lyakhov daniil-lyakhov force-pushed the dl/fx/model_transformer_tests_actual branch from 6506139 to 584bc48 Compare October 15, 2024 15:49
@AlexanderDokuchaev AlexanderDokuchaev merged commit 5619afb into openvinotoolkit:develop Oct 16, 2024
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@AlexanderDokuchaev AlexanderDokuchaev AlexanderDokuchaev approved these changes

Assignees
No one assigned
Labels
experimental NNCF PT Pull requests that updates NNCF PyTorch NNCF PTQ Pull requests that updates NNCF PTQ
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@daniil-lyakhov @AlexanderDokuchaev