Skip to content

Dtype selective build: fail if not xplat, if portable/optimized not in kernel_deps #10985

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

Closed
wants to merge 1 commit into from

Conversation

lucylq
Copy link
Contributor

@lucylq lucylq commented May 19, 2025

Copy link

pytorch-bot bot commented May 19, 2025

🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/executorch/10985

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

❌ 8 New Failures, 46 Unrelated Failures

As of commit 55e0507 with merge base 9663bfb (image):

NEW FAILURES - The following jobs have failed:

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

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

lucylq added a commit that referenced this pull request May 19, 2025
…n kernel_deps

Differential Revision: [D75027794](https://our.internmc.facebook.com/intern/diff/D75027794/)

ghstack-source-id: 284959143
Pull Request resolved: #10985
@facebook-github-bot facebook-github-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 May 19, 2025
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D75027794

Copy link

This PR needs a release notes: label

If your change should be included in the release notes (i.e. would users of this library care about this change?), please use a label starting with release notes:.

If not, please add the release notes: none label.

To add a label, you can comment to pytorchbot, for example
@pytorchbot label "release notes: none"

For more information, see
https://github.com/pytorch/pytorch/wiki/PyTorch-AutoLabel-Bot#why-categorize-for-release-notes-and-how-does-it-work.

lucylq added a commit that referenced this pull request May 19, 2025
…n kernel_deps

#10985

Try to make user error harder for dtype selective build

Differential Revision: [D75027794](https://our.internmc.facebook.com/intern/diff/D75027794/)

[ghstack-poisoned]
lucylq added a commit that referenced this pull request May 19, 2025
…n kernel_deps

#10985

Try to make user error harder for dtype selective build

Differential Revision: [D75027794](https://our.internmc.facebook.com/intern/diff/D75027794/)

ghstack-source-id: 284970179
Pull Request resolved: #10988
lucylq added a commit that referenced this pull request May 19, 2025
…n kernel_deps

#10985

Try to make user error harder for dtype selective build.

Emit warning for now, as too many failures when set to failure :(
For example:
```
buck2 build //xplat/sgr/resources/tests/handwriting:pkg
buck2 build fbsource//xplat/sgr/resources/mwa:main_pkg_libAndroid

Differential Revision: [D75027794](https://our.internmc.facebook.com/intern/diff/D75027794/)

[ghstack-poisoned]
lucylq added a commit that referenced this pull request May 19, 2025
…n kernel_deps

#10985

Try to make user error harder for dtype selective build.

Emit warning for now, as too many failures when set to failure :(
For example:
```
buck2 build //xplat/sgr/resources/tests/handwriting:pkg
buck2 build fbsource//xplat/sgr/resources/mwa:main_pkg_libAndroid

Differential Revision: [D75027794](https://our.internmc.facebook.com/intern/diff/D75027794/)

ghstack-source-id: 284982140
Pull Request resolved: #10990
lucylq added a commit that referenced this pull request May 20, 2025
…if portable/optimized not in kernel_deps"

#10985

Try to make user error harder for dtype selective build.

Emit warning for now, as too many failures when set to failure :(
For example:
```
buck2 build //xplat/sgr/resources/tests/handwriting:pkg
buck2 build fbsource//xplat/sgr/resources/mwa:main_pkg_libAndroid

Differential Revision: [D75027794](https://our.internmc.facebook.com/intern/diff/D75027794/)

[ghstack-poisoned]
lucylq added a commit that referenced this pull request May 20, 2025
…mized not in kernel_deps"

#10985

Try to make user error harder for dtype selective build.

Emit warning for now, as too many failures when set to failure :(
For example:
```
buck2 build //xplat/sgr/resources/tests/handwriting:pkg
buck2 build fbsource//xplat/sgr/resources/mwa:main_pkg_libAndroid

Differential Revision: [D75027794](https://our.internmc.facebook.com/intern/diff/D75027794/)

[ghstack-poisoned]
lucylq added a commit that referenced this pull request May 20, 2025
Pull Request resolved: #10990

#10985

Try to make user error harder for dtype selective build.

Currently, if dtype selective build cannot be enabled we silently fallback to regular dtype selective build.

This diff:
1. Emits failure when there is nothing in kernel_deps and dtype_selective_build=True.
2. Remove dtype_selective_build=True for cases in (1).
3. Warning that dtype selective build is not enabled if user is not in xplat.

ghstack-source-id: 285196226

Differential Revision: [D75027794](https://our.internmc.facebook.com/intern/diff/D75027794/)
lucylq added a commit that referenced this pull request May 20, 2025
…if portable/optimized not in kernel_deps"

#10985

Try to make user error harder for dtype selective build.

Emit warning for now, as too many failures when set to failure :(
For example:
```
buck2 build //xplat/sgr/resources/tests/handwriting:pkg
buck2 build fbsource//xplat/sgr/resources/mwa:main_pkg_libAndroid

Differential Revision: [D75027794](https://our.internmc.facebook.com/intern/diff/D75027794/)

[ghstack-poisoned]
lucylq added a commit that referenced this pull request May 20, 2025
…mized not in kernel_deps"

#10985

Try to make user error harder for dtype selective build.

Emit warning for now, as too many failures when set to failure :(
For example:
```
buck2 build //xplat/sgr/resources/tests/handwriting:pkg
buck2 build fbsource//xplat/sgr/resources/mwa:main_pkg_libAndroid

Differential Revision: [D75027794](https://our.internmc.facebook.com/intern/diff/D75027794/)

[ghstack-poisoned]
@lucylq lucylq closed this May 20, 2025
@lucylq lucylq had a problem deploying to cherry-pick-bot May 20, 2025 22:44 — with GitHub Actions Failure
kirklandsign pushed a commit that referenced this pull request May 21, 2025
…n kernel_deps (#11020)

Pull Request resolved: #10990

#10985

Try to make user error harder for dtype selective build.

Currently, if dtype selective build cannot be enabled we silently fallback to regular dtype selective build.

This diff:
1. Emits failure when there is nothing in kernel_deps and dtype_selective_build=True.
2. Remove dtype_selective_build=True for cases in (1).
3. Warning that dtype selective build is not enabled if user is not in xplat.

ghstack-source-id: 285201952

Differential Revision: [D75027794](https://our.internmc.facebook.com/intern/diff/D75027794/)
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.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants