Skip to content

Fix the print#11944

Merged
yangw-dev merged 5 commits into
mainfrom
fixBugPrint
Jun 25, 2025
Merged

Fix the print#11944
yangw-dev merged 5 commits into
mainfrom
fixBugPrint

Conversation

@yangw-dev

@yangw-dev yangw-dev commented Jun 24, 2025 •
edited
Loading

Copy link
Copy Markdown
Contributor

clean up the print from common.ts

add --debug flag to execuTorch script.

@yangw-dev
setup link
19ac5a9 
Signed-off-by: Yang Wang <elainewy@meta.com>
@yangw-dev yangw-dev requested a review from huydhn June 24, 2025 23:14
@pytorch-bot

pytorch-bot Bot commented Jun 24, 2025 •
edited
Loading

Copy link
Copy Markdown

🔗 Helpful Links

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

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

❌ 1 New Failure, 1 Cancelled Job

As of commit d8f2bb7 with merge base 5335eeb (image):

NEW FAILURE - The following job has failed:

CANCELLED JOB - The following job was cancelled. Please retry:

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

@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 Jun 24, 2025
@github-actions

github-actions Bot commented Jun 24, 2025

Copy link
Copy Markdown

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:. This helps us keep track and include your important work in the next release notes.

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.

yangw-dev added 2 commits June 24, 2025 16:23
@yangw-dev
setup link
b45d737 
Signed-off-by: Yang Wang <elainewy@meta.com>
@yangw-dev
setup link
544977d 
Signed-off-by: Yang Wang <elainewy@meta.com>
huydhn
huydhn reviewed Jun 24, 2025
View reviewed changes
Comment thread .ci/scripts/benchmark_tooling/analyze_benchmark_stability.py
@yangw-dev yangw-dev requested a review from guangy10 June 25, 2025 00:20
yangw-dev added 2 commits June 24, 2025 22:04
@yangw-dev
Merge branch 'main' into fixBugPrint
955230a
@yangw-dev
setup link
d8f2bb7 
Signed-off-by: Yang Wang <elainewy@meta.com>
guangy10
guangy10 reviewed Jun 25, 2025
View reviewed changes

@guangy10 guangy10 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.

Can I enable the log if I set verbose in the analysis script?

@yangw-dev yangw-dev merged commit 3666680 into main Jun 25, 2025
191 of 193 checks passed
@yangw-dev yangw-dev deleted the fixBugPrint branch June 25, 2025 07:09
hinriksnaer pushed a commit to hinriksnaer/executorch that referenced this pull request Jun 26, 2025
@yangw-dev @hinriksnaer
Fix the print (pytorch#11944)
7d913bc 
clean up the print from common.ts 

 add --debug flag to execuTorch script.

---------

Signed-off-by: Yang Wang <elainewy@meta.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@guangy10 guangy10 guangy10 approved these changes

@huydhn huydhn huydhn 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.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@yangw-dev @huydhn @guangy10 @facebook-github-bot