Skip to content

Address various warnings as errors - #8581

Merged
JakeStevens merged 1 commit into
pytorch:mainfrom
JakeStevens:export-D69139962
Mar 3, 2025
Merged

Address various warnings as errors#8581
JakeStevens merged 1 commit into
pytorch:mainfrom
JakeStevens:export-D69139962

Conversation

@JakeStevens

Copy link
Copy Markdown
Contributor

Summary:
Some projects uses more restrictive build options than currently used in ET CI.

This means we encountered a number of errors when enabling for a microcontroller.

Reviewed By: digantdesai, swolchok

Differential Revision: D69139962

@pytorch-bot

pytorch-botBot commented Feb 19, 2025

Copy link
Copy Markdown

🔗 Helpful Links

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

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

⏳ No Failures, 1 Pending

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

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

@facebook-github-botfacebook-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 Feb 19, 2025
@facebook-github-bot

Copy link
Copy Markdown
Contributor

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

@JakeStevens

Copy link
Copy Markdown
ContributorAuthor

@mergennachin

mergennachin commented Feb 20, 2025

Copy link
Copy Markdown
Contributor

@JakeStevens

A few remaining errors in OSS CI: hhttps://github.com/pytorch/executorch/actions/runs/13422197087/job/37540258041

and also lint error:

https://github.com/pytorch/executorch/actions/runs/13422197104/job/37540247065?pr=8581

@facebook-github-bot

Copy link
Copy Markdown
Contributor

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

JakeStevens added a commit to JakeStevens/executorch that referenced this pull request Feb 20, 2025
Summary:
Pull Request resolved: pytorch#8581
Some projects uses more restrictive build options than currently used in ET CI.
This means we encountered a number of errors when enabling for a microcontroller.
Reviewed By: digantdesai, swolchok
Differential Revision: D69139962
@JakeStevens

Copy link
Copy Markdown
ContributorAuthor

@pytorchbot label "topic: not user facing

@pytorch-bot

Copy link
Copy Markdown

❌ 🤖 pytorchbot command failed:

Got EOF while in a quoted string```
Try `@pytorchbot --help` for more info.

@facebook-github-bot

Copy link
Copy Markdown
Contributor

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

JakeStevens added a commit to JakeStevens/executorch that referenced this pull request Feb 20, 2025
Summary:
Pull Request resolved: pytorch#8581
Some projects uses more restrictive build options than currently used in ET CI.
This means we encountered a number of errors when enabling for a microcontroller.
Reviewed By: digantdesai, swolchok
Differential Revision: D69139962
@facebook-github-bot

Copy link
Copy Markdown
Contributor

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

JakeStevens added a commit to JakeStevens/executorch that referenced this pull request Feb 20, 2025
Summary:
Pull Request resolved: pytorch#8581
Some projects uses more restrictive build options than currently used in ET CI.
This means we encountered a number of errors when enabling for a microcontroller.
Reviewed By: digantdesai, swolchok
Differential Revision: D69139962
@facebook-github-bot

Copy link
Copy Markdown
Contributor

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

JakeStevens added a commit to JakeStevens/executorch that referenced this pull request Feb 20, 2025
Summary:
Pull Request resolved: pytorch#8581
Some projects uses more restrictive build options than currently used in ET CI.
This means we encountered a number of errors when enabling for a microcontroller.
Reviewed By: digantdesai, swolchok
Differential Revision: D69139962
@facebook-github-bot

Copy link
Copy Markdown
Contributor

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

JakeStevens added a commit to JakeStevens/executorch that referenced this pull request Feb 21, 2025
Summary:
Pull Request resolved: pytorch#8581
Some projects uses more restrictive build options than currently used in ET CI.
This means we encountered a number of errors when enabling for a microcontroller.
Reviewed By: digantdesai, swolchok
Differential Revision: D69139962
@facebook-github-bot

Copy link
Copy Markdown
Contributor

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

JakeStevens added a commit to JakeStevens/executorch that referenced this pull request Feb 21, 2025
Summary:
Pull Request resolved: pytorch#8581
Some projects uses more restrictive build options than currently used in ET CI.
This means we encountered a number of errors when enabling for a microcontroller.
Reviewed By: digantdesai, swolchok
Differential Revision: D69139962
@facebook-github-bot

Copy link
Copy Markdown
Contributor

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

JakeStevens added a commit to JakeStevens/executorch that referenced this pull request Feb 22, 2025
Summary:
Pull Request resolved: pytorch#8581
Some projects uses more restrictive build options than currently used in ET CI.
This means we encountered a number of errors when enabling for a microcontroller.
Reviewed By: digantdesai, swolchok
Differential Revision: D69139962
@facebook-github-bot

Copy link
Copy Markdown
Contributor

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

JakeStevens added a commit to JakeStevens/executorch that referenced this pull request Feb 22, 2025
Summary:
Pull Request resolved: pytorch#8581
Some projects uses more restrictive build options than currently used in ET CI.
This means we encountered a number of errors when enabling for a microcontroller.
Reviewed By: digantdesai, swolchok
Differential Revision: D69139962
@facebook-github-bot

Copy link
Copy Markdown
Contributor

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

JakeStevens added a commit to JakeStevens/executorch that referenced this pull request Feb 22, 2025
Summary:
Pull Request resolved: pytorch#8581
Some projects uses more restrictive build options than currently used in ET CI.
This means we encountered a number of errors when enabling for a microcontroller.
Reviewed By: digantdesai, swolchok
Differential Revision: D69139962
@facebook-github-bot

Copy link
Copy Markdown
Contributor

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

JakeStevens added a commit to JakeStevens/executorch that referenced this pull request Feb 22, 2025
Summary:
Pull Request resolved: pytorch#8581
Some projects uses more restrictive build options than currently used in ET CI.
This means we encountered a number of errors when enabling for a microcontroller.
Reviewed By: digantdesai, swolchok
Differential Revision: D69139962
@facebook-github-bot

Copy link
Copy Markdown
Contributor

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

JakeStevens added a commit to JakeStevens/executorch that referenced this pull request Feb 22, 2025
Summary:
Pull Request resolved: pytorch#8581
Some projects uses more restrictive build options than currently used in ET CI.
This means we encountered a number of errors when enabling for a microcontroller.
Reviewed By: digantdesai, swolchok
Differential Revision: D69139962
@facebook-github-bot

Copy link
Copy Markdown
Contributor

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

@mergennachin

Copy link
Copy Markdown
Contributor

Summary:
Pull Request resolved: pytorch#8581
Some projects uses more restrictive build options than currently used in ET CI.
This means we encountered a number of errors when enabling for a microcontroller.
Reviewed By: digantdesai, swolchok
Differential Revision: D69139962
@facebook-github-bot

Copy link
Copy Markdown
Contributor

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

@mergennachinmergennachin left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thank you so much!

@JakeStevens
JakeStevens merged commit ff36efd into pytorch:mainMar 3, 2025
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ciflow/trunkCLA SignedThis label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.fb-exportedtopic: not user facing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@JakeStevens@facebook-github-bot@mergennachin@digantdesai