-
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
[HOTFIX][TARGET] Change LOG in compilation config to DLOG #9486
Conversation
CompilationConfig was merged in on the basis that it is an internal experimental structure that helps to group the target. Constructing the config should not emit messages for most cases. Change LOG(INFO) to DLOG(INFO) so users won't be overwhelmed by messages. There are a few warning cases that also changes to DLOG. Given CompilationConfig is still experimental, it would be better to respect the current default convention and not trigger warnings that indicate an non-experimental suggestion. Warnings can be updated according with the convention.
Fixes #9487 cc @mbs-octoml @zxybazh @Lunderberg @csullivan @masahi |
cc @junrushao1994 this needs to go into v0.8 release |
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 this @tqchen, this should definitely not be happening 😸
sorry for the trouble -- i'll assume log messages are part of the api and not get so trigger happy. |
Retriggered the CI upon failure |
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, thanks TQ for the fix.
cc @Mousius a weird CI error from Can you take a look at what's going on? Looks like #9480 didn't fix the problem? |
i just re-re-triggered the CI and let's see how it goes. |
The CI doesn't fail this time. Thanks for the quick fix! |
* main: (119 commits) [Topi][Op][PyTorch][Vitas] Fix inconsistent kernel layout conventions for conv2d_transpose (apache#9336) Fix repository URL in ubuntu_install_rocm.sh (apache#9425) Add LLVM-13 installation to Docker setup (apache#9498) [Relay] Use target_host determined at Relay level instead of recalculating it (apache#9499) Arm(R) Ethos(TM)-U NPU BinaryElementwise operators support (apache#9442) [COMMUNITY] Junru's and Wuwei's PGP key for ASF release (apache#9488) Add default for split op (apache#9489) [HOTFIX][TARGET] Change LOG in compilation config to DLOG (apache#9486) Fixed some warnings about lambda's closures that are bigger than necessary (apache#9481) [Support] Add libinfo into the runtime build (apache#9310) Change Call with TIRCallAttrs to call_lowered op (apache#9312) [ETHOSN] Streamline Ethos(TM)-N cross-compile rpc usage (apache#9477) [CMSIS-NN] Assert correct amount of CMSIS-NN artifacts in MLF (apache#9480) [MicroTVM][PyTest] Explicitly skip MicroTVM unittests. (apache#9335) [microNPU] Replace ICHECK with diagnostic context in type inference (apache#9470) Better host handling in CompilationConfig & debug printing (apache#9460) [AOT][Tests] Use pre-built libraries in Reference System tests (apache#9271) [TIR] Add type hint for TIR (apache#9432) [TVMC] Add test for quantized pytorch model (apache#9467) [CMSIS-NN] Convert CMSIS-NN to use Target Hooks (apache#9397) ...
CompilationConfig was merged in on the basis that it is an internal experimental structure that helps to group the target. Constructing the config should not emit messages for most cases. Change LOG(INFO) to DLOG(INFO) so users won't be overwhelmed by messages. There are a few warning cases that also changes to DLOG. Given CompilationConfig is still experimental, it would be better to respect the current default convention and not trigger warnings that indicate an non-experimental suggestion. Warnings can be updated according with the convention.
CompilationConfig was merged in on the basis that it is an internal experimental structure that helps to group the target. Constructing the config should not emit messages for most cases. Change LOG(INFO) to DLOG(INFO) so users won't be overwhelmed by messages. There are a few warning cases that also changes to DLOG. Given CompilationConfig is still experimental, it would be better to respect the current default convention and not trigger warnings that indicate an non-experimental suggestion. Warnings can be updated according with the convention.
CompilationConfig was merged in on the basis that it is an internal experimental structure that helps to group the target. Constructing the config should not emit messages for most cases. Change LOG(INFO) to DLOG(INFO) so users won't be overwhelmed by messages. There are a few warning cases that also changes to DLOG. Given CompilationConfig is still experimental, it would be better to respect the current default convention and not trigger warnings that indicate an non-experimental suggestion. Warnings can be updated according with the convention.
CompilationConfig was merged in on the basis that it is an internal experimental structure that helps to group the target. Constructing the config should not emit messages for most cases. Change LOG(INFO) to DLOG(INFO) so users won't be overwhelmed by messages. There are a few warning cases that also changes to DLOG. Given CompilationConfig is still experimental, it would be better to respect the current default convention and not trigger warnings that indicate an non-experimental suggestion. Warnings can be updated according with the convention.
CompilationConfig was merged in on the basis that it is an internal experimental structure that helps to group the target. Constructing the config should not emit messages for most cases. Change LOG(INFO) to DLOG(INFO) so users won't be overwhelmed by messages. There are a few warning cases that also changes to DLOG. Given CompilationConfig is still experimental, it would be better to respect the current default convention and not trigger warnings that indicate an non-experimental suggestion. Warnings can be updated according with the convention.
CompilationConfig was merged in on the basis that it is an internal experimental
structure that helps to group the target. Constructing the config should not
emit messages for most cases. Change LOG(INFO) to DLOG(INFO)
so users won't be overwhelmed by messages.
There are a few warning cases that also changes to DLOG. Given CompilationConfig
is still experimental, it would be better to respect the current default convention
and not trigger warnings that indicate an non-experimental suggestion.
Warnings can be updated according to the convention as it evolves.