-
Notifications
You must be signed in to change notification settings - Fork 517
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
Disable LTC and update PyTorch #1472
Conversation
When built using PyTorch's master branch, the LTC code has been failing to build for a few days. As a result, the PyTorch version referenced by Torch-MLIR is stalled to the one from October 4th. In an effort to advance to PyTorch version, this patch disables LTC, and a subsequent patch will advance the PyTorch version.
@pashu123 It seems like after updating the PyTorch version, one of your tests fails because the oracle is slightly different from the computed result. Do you have a hunch as to why this might be the case?
|
if required we can xfail to roll it forward with an issue tracker |
Also disables the `UpSampleNearest2dDynamicFactor_basic` e2e test, since the (PyTorch) oracle differs from the computed value for both the refbackend and the eager_mode backends.
Disabled the test for now, and noted it in this issue: #1473. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@antoniojkim We want to try and get tonights build successful. Happy to enable LTC again once it is fixed.
@pashu123 please re-enable the tests once it is fixed. |
Thanks, @ashay for filing it as an issue. I will take a look at it. |
When built using PyTorch's master branch, the LTC code has been failing
to build for a few days. As a result, the PyTorch version referenced by
Torch-MLIR is stalled to the one from October 4th.
This PR contains two patches: the first disables LTC and the second
advances the PyTorch version.