Skip to content

Quiet Inductor #135521 #155450

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Quiet Inductor #135521 #155450

wants to merge 1 commit into from

Conversation

hardikkgupta
Copy link

@hardikkgupta hardikkgupta commented Jun 9, 2025 •
edited by pytorch-bot bot
Loading

Fixes #135521

File Change Notes
torch/_inductor/test_case.py * Added module logger log = logging.getLogger(name). * Wrapped counter dump in if os.getenv("INDUCTOR_COUNTER_LOGS") or log.isEnabledFor(logging.DEBUG): .... * Replaced print() with log.debug("%s %s", k, v.most_common()). Opt-in via env flag or logging.basicConfig(level=logging.DEBUG).
test/test_log_clean.py New unit-test that 1) launches a subprocess using TestCase, 2) asserts default stdout is empty, 3) returns non-zero on failure (captured by pytest). Guards against regression.

@pytorchbot label "topic: not user facing"

cc @voznesenskym @penguinwu @EikanWang @jgong5 @Guobing-Chen @XiaobingSuper @zhuhaozhe @blzheng @wenzhe-nrv @jiayisunx @ipiszy @chenyang78 @kadeng @muchulee8 @amjames @chauhang @aakhundov @coconutruben

@pytorch-bot PyTorch Bot
Copy link

pytorch-bot bot commented Jun 9, 2025 •
edited
Loading

🔗 Helpful Links

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

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

✅ No Failures

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

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

@linux-foundation-easycla Linux Foundation: EasyCLA
Copy link

linux-foundation-easycla bot commented Jun 9, 2025 •
edited
Loading

CLA Signed


The committers listed above are authorized under a signed CLA.

@hardikkgupta
Copy link
Author

@pytorchbot label "topic: not user facing"

@pytorch-bot pytorch-bot bot added the topic: not user facing topic category label Jun 9, 2025
@albanD albanD requested a review from eellison June 10, 2025 13:40
@albanD albanD added the triaged This issue has been looked at a team member, and triaged and prioritized into an appropriate module label Jun 10, 2025
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Aug 9, 2025

Looks like this PR hasn't been updated in a while so we're going to go ahead and mark this as Stale.
Feel free to remove the Stale label if you feel this was a mistake.
If you are unable to remove the Stale label please contact a maintainer in order to do so.
If you want the bot to never mark this PR stale again, add the no-stale label.
Stale pull requests will automatically be closed after 30 days of inactivity.

@github-actions github-actions bot added the Stale label Aug 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@eellison eellison Awaiting requested review from eellison

Assignees
No one assigned
Labels
module: inductor open source Stale topic: not user facing topic category triaged This issue has been looked at a team member, and triaged and prioritized into an appropriate module
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

reduce torch.compile default logs
3 participants
@hardikkgupta @albanD @pytorchbot