-
Notifications
You must be signed in to change notification settings - Fork 3.4k
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
[RELAY] Fix the FoldConstant Regression for VTA #6377
Conversation
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.
Thanks for the fix. I can confirm that this patch causes the VTA end to end test to pass locally.
For reference here's the PR that introduced the regression. The regression was not caught because the of a buggy CI that did not catch the regression, until recently when Tianqi fixed the CI script. |
Thanks @tqchen for the fix. Thanks @tmoreau89 for raising this. The fix ensures that the compilation time improvement is retained from the origin PR #6195 |
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.
I see...LGTM!
This PR fixes an error guard during the documentation build step. - Temporary disables VTA frontend tutorial due to the regression of deploy_detection
@@ -230,7 +216,17 @@ class ConstantFolder : public ExprMutator { | |||
auto entry_func = Downcast<Function>(mod->Lookup("main")); | |||
expr = expr.as<FunctionNode>() == nullptr ? entry_func->body : entry_func; | |||
|
|||
FInterpreter executor = GetInterpreter(mod); | |||
using tvm::transform::PassContext; |
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.
Why inline this back in?
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.
We will need the make sure the context is set during the executor(input)
call besides the creation.
* [RELAY] Fix the FoldConstant Regression for VTA * [CI] Fix error guard that was missed in VTA. This PR fixes an error guard during the documentation build step. - Temporary disables VTA frontend tutorial due to the regression of deploy_detection
* [RELAY] Fix the FoldConstant Regression for VTA * [CI] Fix error guard that was missed in VTA. This PR fixes an error guard during the documentation build step. - Temporary disables VTA frontend tutorial due to the regression of deploy_detection
* [RELAY] Fix the FoldConstant Regression for VTA * [CI] Fix error guard that was missed in VTA. This PR fixes an error guard during the documentation build step. - Temporary disables VTA frontend tutorial due to the regression of deploy_detection
cc @tmoreau89 @anijain2305