Location via proxy:   [ UP ]  
[Report a bug]   [Manage cookies]                
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

[Release][TorchFX] Export to torch.export.export_for_training #3087

Merged

Conversation

daniil-lyakhov
Copy link
Collaborator

PR #3075 to the release branch:

Changes

  • TorchFX Unit tests are moved from torch._export.capture_pre_autograd_graph to
    torch.export.export_for_training
    ALL REFERENCE GRAPHS WERE VALIDATED MANUALLY
  • BC types for fuse_bn_node are updated
  • NNCFGraphBuilder is updated to support a batch-norm type with only one output node (instead of three)
  • Model extractor does not traverse down from constans to prevent redundant nodes in the extracted model when the constant is shared
  • shared_constants_unification_transformation is removed
  • Tests which require capture_pre_autograd_graph are removed

Reason for changes

  • To migrate to the lates and recommended export method for TorchFX backend

Related tickets

#2766

Tests

test_shared_constants_unification_not_connected_const post_training_quantization/540/ is finished successfully

@daniil-lyakhov daniil-lyakhov requested a review from a team as a code owner November 14, 2024 12:27
@github-actions github-actions bot added NNCF PT Pull requests that updates NNCF PyTorch experimental release target labels Nov 14, 2024
…it#3075)

### Changes

* TorchFX Unit tests are moved from
`torch._export.capture_pre_autograd_graph` to
`torch.export.export_for_training`
ALL REFERENCE GRAPHS WERE VALIDATED MANUALLY 
* BC types for `fuse_bn_node` are updated
* NNCFGraphBuilder is updated to support a batch-norm type with only one
output node (instead of three)
* Model extractor does not traverse down from constans to prevent
redundant nodes in the extracted model when the constant is shared
* `shared_constants_unification_transformation` is removed
* Tests which require `capture_pre_autograd_graph` are removed

### Reason for changes

* To migrate to the lates and recommended export method for TorchFX
backend

### Related tickets

openvinotoolkit#2766 

### Tests

test_shared_constants_unification_not_connected_const
post_training_quantization/540/ is finished successfully
@daniil-lyakhov daniil-lyakhov changed the title [Release][TorchFX] Export to torch.export.export_for_training [Release][TorchFX] Export to torch.export.export_for_training Nov 14, 2024
@KodiaqQ KodiaqQ merged commit e269fab into openvinotoolkit:release_v2140 Nov 14, 2024
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
experimental NNCF PT Pull requests that updates NNCF PyTorch release target
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants