Skip to content
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.

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

Add the torch.compile executor as a test executor #299

Open
carmocca opened this issue Apr 29, 2024 · 1 comment
Open

Add the torch.compile executor as a test executor #299

carmocca opened this issue Apr 29, 2024 · 1 comment
Labels

Comments

@carmocca
Copy link
Contributor

carmocca commented Apr 29, 2024

🚀 Feature

Motivation

Our op coverage for the torch.compile executor could be better, currently we only add it in the partial from in

else set(_all_test_executors() + [TorchCompileCatExecutor])

Pitch

Add the TorchCompileCatExecutor and TorchCompileExecutor test executors to

executors = [TorchExecutor]
and work through the errors that pop up in CI

Additional context

Requires #140 to land first.

cc @apaz-cli

@mruberry
Copy link
Collaborator

mruberry commented May 6, 2024

triage review — what would be the increase in e2e test time for add this? Should we consider some additional mechanisms of reducing CI time before increasing it?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants