Skip to content

Remove marlin_kernel_nm.cu#3613

Merged
howardzhang-cv merged 9 commits intomainfrom
gh/howardzhang-cv/3/head
Jan 13, 2026
Merged

Remove marlin_kernel_nm.cu#3613
howardzhang-cv merged 9 commits intomainfrom
gh/howardzhang-cv/3/head

Conversation

@howardzhang-cv
Copy link
Copy Markdown
Contributor

@howardzhang-cv howardzhang-cv commented Jan 9, 2026 •
edited
Loading

Stack from ghstack (oldest at bottom):

Summary: Removed marlin_kernel_nm.cu plus associated support files/functions
Removed associated layouts and tensor implementations
Removed tests and benchmarks for marlin sparse kernel
Removed dead code from deletion

Related to issue #3516

Differential Revision: D90410991

@howardzhang-cv
Remove marlin_kernel_nm.cu
1d5198e 
Summary: Removed marlin_kernel_nm.cu plus associated support files/functions
Removed associated layouts and tensor implementations
Removed tests and benchmarks for marlin sparse kernel
Removed dead code from deletion

Related to issue #3516

[ghstack-poisoned]
@pytorch-bot
Copy link
Copy Markdown

pytorch-bot bot commented Jan 9, 2026 •
edited
Loading

🔗 Helpful Links

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

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

✅ No Failures

As of commit d91e78d with merge base ab80e93 (image):
💚 Looks good so far! There are no failures yet. 💚

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

howardzhang-cv added a commit that referenced this pull request Jan 9, 2026
@howardzhang-cv
Remove marlin_kernel_nm.cu
5753d6c 
Summary: Removed marlin_kernel_nm.cu plus associated support files/functions
Removed associated layouts and tensor implementations
Removed tests and benchmarks for marlin sparse kernel
Removed dead code from deletion

Related to issue #3516

ghstack-source-id: 284c823
Pull Request resolved: #3613
@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 Jan 9, 2026
@howardzhang-cv howardzhang-cv added module: deprecation Use this tag if this PR deprecates a feature module: bc-breaking Use this tag if this PR breaks backward compatibility labels Jan 9, 2026
@howardzhang-cv
Copy link
Copy Markdown
Contributor Author

howardzhang-cv commented Jan 9, 2026

@howardzhang-cv has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

jerryzh168
jerryzh168 approved these changes Jan 9, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

@jerryzh168 jerryzh168 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please fix ruff, and make sure no related internal failures before merging

@howardzhang-cv
Update
1b01821 
[ghstack-poisoned]
howardzhang-cv added a commit that referenced this pull request Jan 9, 2026
@howardzhang-cv
Remove marlin_kernel_nm.cu
68dceb2 
Summary: Removed marlin_kernel_nm.cu plus associated support files/functions
Removed associated layouts and tensor implementations
Removed tests and benchmarks for marlin sparse kernel
Removed dead code from deletion

Related to issue #3516

ghstack-source-id: 23a416a
Pull Request resolved: #3613
@howardzhang-cv
Copy link
Copy Markdown
Contributor Author

howardzhang-cv commented Jan 9, 2026

@howardzhang-cv has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

1 similar comment
@howardzhang-cv
Copy link
Copy Markdown
Contributor Author

howardzhang-cv commented Jan 13, 2026

@howardzhang-cv has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

@howardzhang-cv
Update (base update)
70fa73d 
[ghstack-poisoned]
@howardzhang-cv
Update
4a47b32 
[ghstack-poisoned]
howardzhang-cv added a commit that referenced this pull request Jan 13, 2026
@howardzhang-cv
Remove marlin_kernel_nm.cu
a94b168 
Summary: Removed marlin_kernel_nm.cu plus associated support files/functions
Removed associated layouts and tensor implementations
Removed tests and benchmarks for marlin sparse kernel
Removed dead code from deletion

Related to issue #3516

ghstack-source-id: 8d90612
Pull Request resolved: #3613
@howardzhang-cv
Copy link
Copy Markdown
Contributor Author

howardzhang-cv commented Jan 13, 2026

@howardzhang-cv has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

@howardzhang-cv howardzhang-cv changed the base branch from gh/howardzhang-cv/3/base to main January 13, 2026 18:15
@howardzhang-cv
Copy link
Copy Markdown
Contributor Author

howardzhang-cv commented Jan 13, 2026

@howardzhang-cv has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

@howardzhang-cv howardzhang-cv merged commit 985d970 into main Jan 13, 2026
62 of 64 checks passed
jcaip pushed a commit that referenced this pull request Jan 22, 2026
@howardzhang-cv @jcaip
Remove marlin_kernel_nm.cu (#3613)
3ac19b6 
* Remove marlin_kernel_nm.cu

Summary: Removed marlin_kernel_nm.cu plus associated support files/functions
Removed associated layouts and tensor implementations
Removed tests and benchmarks for marlin sparse kernel
Removed dead code from deletion

Related to issue #3516

[ghstack-poisoned]

* Update

[ghstack-poisoned]

* Remove PackedLinearInt8DynamicActivationIntxWeightLayout import

* Ruff fix: remove unused import

Removed unused import of compute_max_diff.

* ruff format
@andrewor14 andrewor14 mentioned this pull request Jan 26, 2026
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jerryzh168 jerryzh168 jerryzh168 approved these changes

Assignees

No one assigned

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. module: bc-breaking Use this tag if this PR breaks backward compatibility module: deprecation Use this tag if this PR deprecates a feature

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@howardzhang-cv @jerryzh168