Skip to content

Expose torch.compiler.config.force_disable_caches as a public API#166699

Closed
gmagogsfm wants to merge 1 commit intopytorch:mainfrom
gmagogsfm:add_force_disable_cache_api
Closed

Expose torch.compiler.config.force_disable_caches as a public API#166699
gmagogsfm wants to merge 1 commit intopytorch:mainfrom
gmagogsfm:add_force_disable_cache_api

Conversation

@gmagogsfm
Copy link
Contributor

@gmagogsfm gmagogsfm commented Oct 31, 2025

Exposing this flag as some upstream frameworks (like vLLM) could benefit from knowing whether torch.compile caches are enabled or not to adjust their own caching behavior.

cc @mlazos

@pytorch-bot
Copy link

pytorch-bot bot commented Oct 31, 2025

🔗 Helpful Links

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

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

✅ You can merge normally! (1 Unrelated Failure)

As of commit a0c5b25 with merge base e69aaaf (image):

FLAKY - The following job failed but was likely due to flakiness present on trunk:

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

@gmagogsfm
Copy link
Contributor Author

@pytorchbot merge -r master

@pytorch-bot
Copy link

pytorch-bot bot commented Nov 5, 2025

❌ 🤖 pytorchbot command failed:

@pytorchbot merge: error: argument -r/--rebase: invalid choice: 'master' (choose from 'viable/strict', 'main')

usage: @pytorchbot merge [-f MESSAGE | -i] [-ic] [-r [{viable/strict,main}]]

Try @pytorchbot --help for more info.

@gmagogsfm
Copy link
Contributor Author

@pytorchbot merge -r main

@pytorch-bot pytorch-bot bot added the ciflow/trunk Trigger trunk jobs on your pull request label Nov 5, 2025
@pytorchmergebot
Copy link
Collaborator

@pytorchbot started a rebase job onto refs/remotes/origin/main. Check the current status here

@pytorchmergebot
Copy link
Collaborator

Successfully rebased add_force_disable_cache_api onto refs/remotes/origin/main, please pull locally before adding more changes (for example, via git checkout add_force_disable_cache_api && git pull --rebase)

@pytorchmergebot
Copy link
Collaborator

Merge started

Your change will be merged once all checks pass (ETA 0-4 Hours).

Learn more about merging in the wiki.

Questions? Feedback? Please reach out to the PyTorch DevX Team

Advanced Debugging
Check the merge workflow status
here

@pytorchmergebot
Copy link
Collaborator

Merge failed

Reason: Could not find commit that was pushed before comment 3492444133

Details for Dev Infra team Raised by workflow job

@gmagogsfm
Copy link
Contributor Author

@pytorchbot merge -r main

@pytorchmergebot
Copy link
Collaborator

@pytorchbot started a rebase job onto refs/remotes/origin/main. Check the current status here

@pytorchmergebot
Copy link
Collaborator

Successfully rebased add_force_disable_cache_api onto refs/remotes/origin/main, please pull locally before adding more changes (for example, via git checkout add_force_disable_cache_api && git pull --rebase)

@pytorchmergebot
Copy link
Collaborator

Merge started

Your change will be merged once all checks pass (ETA 0-4 Hours).

Learn more about merging in the wiki.

Questions? Feedback? Please reach out to the PyTorch DevX Team

Advanced Debugging
Check the merge workflow status
here

@pytorchmergebot
Copy link
Collaborator

Merge failed

Reason: Commit bed2aa4 was HEAD when comment 3493698982 was posted but now the latest commit on the PR is a0c5b25. Please re-issue the merge command to merge the latest commit.

Details for Dev Infra team Raised by workflow job

@gmagogsfm
Copy link
Contributor Author

@pytorchbot merge

@pytorchmergebot
Copy link
Collaborator

Merge started

Your change will be merged once all checks pass (ETA 0-4 Hours).

Learn more about merging in the wiki.

Questions? Feedback? Please reach out to the PyTorch DevX Team

Advanced Debugging
Check the merge workflow status
here

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ciflow/trunk Trigger trunk jobs on your pull request Merged release notes: inductor

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants