Skip to content

Remove tutorials/calibration_flow/ and update stale references#4257

Merged
jerryzh168 merged 1 commit intomainfrom
gh/jerryzh168/81/head
Apr 9, 2026
Merged

Remove tutorials/calibration_flow/ and update stale references#4257
jerryzh168 merged 1 commit intomainfrom
gh/jerryzh168/81/head

Conversation

@jerryzh168
Copy link
Copy Markdown
Contributor

@jerryzh168 jerryzh168 commented Apr 9, 2026

Stack from ghstack (oldest at bottom):

Summary:
Remove the calibration_flow tutorial folder (static_quant.py, gptq_like.py,
awq_like.py) which depended on LinearActivationQuantizedTensor and other
deprecated AQT APIs. Update documentation references that pointed to these
deleted files.

Test Plan:

  • Verified no remaining references to calibration_flow/, static_quant.py,
    gptq_like.py, or awq_like.py in the codebase
  • Doc links updated to point to remaining valid resources

Summary:
Remove the calibration_flow tutorial folder (static_quant.py, gptq_like.py,
awq_like.py) which depended on LinearActivationQuantizedTensor and other
deprecated AQT APIs. Update documentation references that pointed to these
deleted files.

Test Plan:
- Verified no remaining references to calibration_flow/, static_quant.py,
  gptq_like.py, or awq_like.py in the codebase
- Doc links updated to point to remaining valid resources

[ghstack-poisoned]
@jerryzh168 jerryzh168 requested review from svekars and vkuzo as code owners April 9, 2026 22:32
@pytorch-bot
Copy link
Copy Markdown

pytorch-bot bot commented Apr 9, 2026

🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/ao/4257

Note: Links to docs will display an error until the docs builds have been completed.

❗ 1 Active SEVs

There are 1 currently active SEVs. If your PR is affected, please view them below:

This comment was automatically generated by Dr. CI and updates every 15 minutes.

@meta-cla meta-cla bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Apr 9, 2026
@jerryzh168 jerryzh168 added the module: not user facing Use this tag if you don't want this PR to show up in release notes label Apr 9, 2026
@andrewor14
Copy link
Copy Markdown
Contributor

@claude review please

@claude
Copy link
Copy Markdown

claude bot commented Apr 9, 2026

Claude finished @andrewor14's task in 1m 5s —— View job


Review of PR #4257

  • Gather context and read the diff
  • Review deleted files for any code that should be preserved
  • Review documentation updates for correctness
  • Verify no remaining stale references in the codebase
  • Post review feedback

This is a clean removal of deprecated tutorial code and the corresponding documentation updates. The changes look good.

Deleted files: All three deleted files (static_quant.py, gptq_like.py, awq_like.py) imported deprecated/deleted APIs (AffineQuantizedTensor, to_affine_quantized_intx, LinearActivationQuantizedTensor, to_linear_activation_quantized, to_weight_tensor_with_linear_activation_scale_metadata), consistent with the AQT deletion in #4245. Each file already had a TODO: migrate off AffineQuantizedTensor comment, confirming these were known to be stale. No reusable code needs to be preserved — the static quantization tutorial content is already captured inline in docs/source/eager_tutorials/static_quantization.rst.

Documentation updates:

  • quantization_overview.rst:144 — Rewording removes the dead link to gptq_like.py and simplifies the sentence to reference Autoround directly. The updated Autoround link now points to blob/main/... which is appropriate for docs.
  • static_quantization.rst:8 — Removes the link to the deleted static_quant.py example script. The sentence reads naturally without it since the tutorial is self-contained.
  • static_quantization.rst:239 — Removes the closing paragraph's reference to the deleted script and the mention of float8 static quantization. This is fine since that float8 example was also in the deleted file.
  • autoround/README.md:41 — Replaces the dead gptq_like.py link with a link to the MultiTensor gist, which is the relevant resource for understanding the flow pattern.

Stale reference check: Grep confirms zero remaining references to calibration_flow/, static_quant.py, gptq_like.py, or awq_like.py in the codebase.

No issues found. LGTM.

@jerryzh168 jerryzh168 changed the base branch from gh/jerryzh168/81/base to main April 9, 2026 22:39
@jerryzh168 jerryzh168 merged commit fee43b5 into main Apr 9, 2026
40 of 57 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. module: not user facing Use this tag if you don't want this PR to show up in release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants