pytorch - ✅(Solved) Fix [CI] [Inductor] Perf Testing for deterministic mode [1 pull requests, 1 participants]

Official PRs (…)
ON THIS PAGE

Recommended Tools

×6

Utilities matched from this issue’s tags and category — try them while you read without losing context.

GitHub issue graph ai analysis

Paste a GitHub issue URL. We fetch that issue, discover linked issues from bodies/comments/timeline, collect linked pull requests, and produce a structured English report.

The report is written in English Markdown for sharing and archival.

Helpful · Quick feedback

Loading…
GitHub stats
pytorch/pytorch#177269Fetched 2026-04-08 00:42:20
View on GitHub
Comments
0
Participants
1
Timeline
87
Reactions
1
Author
Participants
Assignees
Timeline (top)
subscribed ×44mentioned ×24labeled ×7referenced ×6

Fix Action

Fixed

PR fix notes

PR #174718: [inductor] fix allocation with deterministic guard

Description (problem / solution / changelog)

Fixes https://github.com/pytorch/pytorch/issues/174386

Co-authored with claude

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

Changed files

  • test/inductor/test_torchinductor.py (modified, +88/-0)
  • torch/_inductor/codegen/cpp_wrapper_cpu.py (modified, +23/-3)
  • torch/_inductor/codegen/cpp_wrapper_cpu_array_ref.py (modified, +3/-1)
  • torch/_inductor/codegen/wrapper.py (modified, +41/-8)
RAW_BUFFERClick to expand / collapse

🚀 The feature, motivation and pitch

This could either be part of the inductor ci smoketest, or potentially something we track more thoroughly in the dashboard. I think a limited smoketest is probably sufficiently.

Alternatives

No response

Additional context

No response

cc @jerryzh168 @seemethere @malfet @pytorch/pytorch-dev-infra @chauhang @penguinwu @voznesenskym @EikanWang @jgong5 @Guobing-Chen @XiaobingSuper @zhuhaozhe @blzheng @wenzhe-nrv @jiayisunx @ipiszy @kadeng @muchulee8 @amjames @aakhundov @coconutruben @jataylo

extent analysis

Fix Plan

To address the issue, we will implement a limited smoketest as part of the inductor CI.

Steps

  • Create a new test file for the smoketest
  • Write test cases to cover the required functionality
  • Integrate the test with the CI pipeline

Example Code

import unittest
from inductor import Inductor

class TestInductorSmokeTest(unittest.TestCase):
    def test_inductor_smoke_test(self):
        # Initialize the inductor
        inductor = Inductor()
        
        # Test the inductor's functionality
        self.assertTrue(inductor.is_functional())

if __name__ == '__main__':
    unittest.main()

Verification

To verify that the fix worked, run the test and check that it passes. You can also monitor the CI pipeline to ensure that the test is running successfully.

Extra Tips

  • Make sure to update the test cases as the functionality of the inductor changes.
  • Consider adding more test cases to cover different scenarios and edge cases.

Vote matrix · Quick signals

Works
Did the solution work? Tap to confirm.
Easy Fix
Was it a quick fix?
Time Saver
Did it save you time?
Blocking
Was it severely blocking?
Common Issue
Are others likely hitting this too?
Flaky / Intermittent
Is it intermittent?
Verified / Reproducible
Can you reproduce it reliably?
Loading…

Still need to ship something?

×6

Another batch ranked right after the header list — different links, same matching logic.

Back to top recommendations

TRENDING

pytorch - ✅(Solved) Fix [CI] [Inductor] Perf Testing for deterministic mode [1 pull requests, 1 participants]