Skip to content

Add parameter_name support to _nvfp4_weight_only_linear_transform#3977

Open
jcaip wants to merge 6 commits intomainfrom
gh/jcaip/28/head
Open

Add parameter_name support to _nvfp4_weight_only_linear_transform#3977
jcaip wants to merge 6 commits intomainfrom
gh/jcaip/28/head

Conversation

@jcaip
Copy link
Contributor

@jcaip jcaip commented Mar 3, 2026

Stack from ghstack (oldest at bottom):

Add parameter_name keyword-only argument to support FqnToConfig
per-parameter quantization. Replace module.weight reads/writes with
getattr/setattr, and use _module_extra_repr partial pattern.

jcaip added 3 commits March 3, 2026 07:00
Add `parameter_name` keyword-only argument to support FqnToConfig
per-parameter quantization. Replace module.weight reads/writes with
getattr/setattr, and use _module_extra_repr partial pattern.

[ghstack-poisoned]
Add `parameter_name` keyword-only argument to the dynamic quantization
branch (step=None) to support FqnToConfig per-parameter quantization.
PREPARE and CONVERT branches are left unchanged as they are
Linear-specific observer flow.

[ghstack-poisoned]
Add `parameter_name` keyword-only argument to support FqnToConfig
per-parameter quantization. Replace module.weight reads/writes with
getattr/setattr, and use _module_extra_repr partial pattern.

[ghstack-poisoned]
@pytorch-bot
Copy link

pytorch-bot bot commented Mar 3, 2026

🔗 Helpful Links

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

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

❌ 1 New Failure, 4 Unrelated Failures

As of commit 4508095 with merge base 5a5029d (image):

NEW FAILURE - The following job has failed:

FLAKY - The following jobs failed but were likely due to flakiness present on trunk:

BROKEN TRUNK - The following jobs failed but were present on the merge base:

👉 Rebase onto the `viable/strict` branch to avoid these failures

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

jcaip added a commit that referenced this pull request Mar 3, 2026
Add `parameter_name` keyword-only argument to support FqnToConfig
per-parameter quantization. Replace module.weight reads/writes with
getattr/setattr, and use _module_extra_repr partial pattern.

ghstack-source-id: 2afd878
Pull Request resolved: #3977
@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 Mar 3, 2026
jcaip added 2 commits March 3, 2026 07:06
…t_only_linear_transform"

Add `parameter_name` keyword-only argument to support FqnToConfig
per-parameter quantization. Replace module.weight reads/writes with
getattr/setattr, and use _module_extra_repr partial pattern.

[ghstack-poisoned]
…ansform"

Add `parameter_name` keyword-only argument to support FqnToConfig
per-parameter quantization. Replace module.weight reads/writes with
getattr/setattr, and use _module_extra_repr partial pattern.

[ghstack-poisoned]
jcaip added a commit that referenced this pull request Mar 3, 2026
Add `parameter_name` keyword-only argument to support FqnToConfig
per-parameter quantization. Replace module.weight reads/writes with
getattr/setattr, and use _module_extra_repr partial pattern.

ghstack-source-id: c00b1a0
Pull Request resolved: #3977
@jcaip jcaip added module: not user facing Use this tag if you don't want this PR to show up in release notes module: inference quantize_ api inference flow labels Mar 3, 2026
@jcaip jcaip changed the base branch from gh/jcaip/28/base to main March 5, 2026 22:37
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: inference quantize_ api inference flow 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