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

[TorchFX][Conformance] Move all models to export_for_training #3078

Conversation

daniil-lyakhov
Copy link
Collaborator

@daniil-lyakhov daniil-lyakhov commented Nov 12, 2024

Changes

All capture_pre_autograd_graph calls in the conformance test were replaced by torch.export.export_for_training.

Reason for changes

To remove deprecated capture_pre_autograd_graph from the conformance test.

Related tickets

#2766

Tests

post_training_quantization/555/ have finished succesfully

@github-actions github-actions bot added NNCF PT Pull requests that updates NNCF PyTorch experimental NNCF PTQ Pull requests that updates NNCF PTQ labels Nov 12, 2024
@daniil-lyakhov daniil-lyakhov changed the title [DO NOT MERGE][TorchFX][Conformance] Move all models to exporte_for_training [DO NOT MERGE][TorchFX][Conformance] Move all models to export_for_training Nov 12, 2024
@daniil-lyakhov daniil-lyakhov force-pushed the dl/fx/conformance_export_for_training branch 3 times, most recently from dd62164 to 69d7508 Compare November 13, 2024 15:12
@daniil-lyakhov daniil-lyakhov force-pushed the dl/fx/conformance_export_for_training branch from 39afba8 to cc92946 Compare November 22, 2024 12:49
@github-actions github-actions bot removed NNCF PT Pull requests that updates NNCF PyTorch experimental labels Nov 22, 2024
@daniil-lyakhov daniil-lyakhov changed the title [DO NOT MERGE][TorchFX][Conformance] Move all models to export_for_training [TorchFX][Conformance] Move all models to export_for_training Nov 22, 2024
@daniil-lyakhov daniil-lyakhov force-pushed the dl/fx/conformance_export_for_training branch from cc92946 to b1a16a9 Compare November 25, 2024 10:16
@daniil-lyakhov daniil-lyakhov marked this pull request as ready for review November 25, 2024 18:09
@daniil-lyakhov daniil-lyakhov requested a review from a team as a code owner November 25, 2024 18:09
@alexsu52 alexsu52 merged commit f61aa89 into openvinotoolkit:develop Nov 26, 2024
14 checks passed
daniil-lyakhov added a commit to daniil-lyakhov/nncf that referenced this pull request Dec 2, 2024
…notoolkit#3078)

### Changes

All `capture_pre_autograd_graph` calls in the conformance test were
replaced by `torch.export.export_for_training`.

### Reason for changes

To remove deprecated `capture_pre_autograd_graph` from the conformance
test.

### Related tickets

openvinotoolkit#2766 

### Tests

post_training_quantization/555/ have finished succesfully
daniil-lyakhov added a commit to daniil-lyakhov/nncf that referenced this pull request Dec 2, 2024
…notoolkit#3078)

### Changes

All `capture_pre_autograd_graph` calls in the conformance test were
replaced by `torch.export.export_for_training`.

### Reason for changes

To remove deprecated `capture_pre_autograd_graph` from the conformance
test.

### Related tickets

openvinotoolkit#2766 

### Tests

post_training_quantization/555/ have finished succesfully
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
NNCF PTQ Pull requests that updates NNCF PTQ
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants