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 compile backend arg for test_set_compiler_fn #5611

Closed

Conversation

vshekhawat-hlab
Copy link
Contributor

Fix the test_set_compiler_fn UT, current test_set_compiler_fn uses inductor as a backend, which is incompatible with HPU. Added backend arg to torch.compile.

Fix the test_set_compiler_fn UT, current test_set_compiler_fn
 uses inductor as a backend, which is incompatible with HPU. Added backend arg to torch.compile.
@loadams loadams requested review from tohtana and removed request for mrwyattii June 4, 2024 16:32
@tohtana
Copy link
Contributor

tohtana commented Jun 4, 2024

Hi, @vshekhawat-hlab,
we are considering to remove set_compiler_fn and its test in #5581. I think we can merge it within this week. Can you wait for a while?

@vshekhawat-hlab
Copy link
Contributor Author

Thanks @tohtana for information.

I am closing this.

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

Successfully merging this pull request may close these issues.

3 participants