-
Notifications
You must be signed in to change notification settings - Fork 35
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
[Tuner] Fix context management #770
Conversation
c79f138
to
8fe8476
Compare
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.
Can you rebase this on top of #771 ?
f668377
to
9eed59b
Compare
There are some local pytest errors and I am working on them, will send another commit after addressing them. |
d7ff2a9
to
139c4dd
Compare
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.
LGTM % one minor issue
Signed-off-by: Bangtian Liu <[email protected]>
Signed-off-by: Bangtian Liu <[email protected]>
Signed-off-by: Bangtian Liu <[email protected]>
Signed-off-by: Bangtian Liu <[email protected]>
Signed-off-by: Bangtian Liu <[email protected]>
Signed-off-by: Bangtian Liu <[email protected]>
a3beb5d
to
f11f214
Compare
This PR is about addressing the MLIR context management issue in the tuner detailed in #764. Although this is a work in progress, I am sending it to gather feedback and ensure I am heading in the right direction. --------- Signed-off-by: Bangtian Liu <[email protected]>
This PR is about addressing the MLIR context management issue in the tuner detailed in #764.
Although this is a work in progress, I am sending it to gather feedback and ensure I am heading in the right direction.