tkonolige commented on pull request #9872: URL: https://github.com/apache/tvm/pull/9872#issuecomment-1007828306
So things are a little more complicated here than it appears at first glance. And error thrown by tvm via the LOG and CHECK infrastructure constructs a TVMError with a backtrace. When this error hits the python/c++ language boundary it is translated into a new python error (the entries in the stacktrace get reordered). However, there is no guarantee that this error will be output to stdout. That means in some cases the current approach you have will not work. I think a better solution is to have the python exception detect if is is being written to a terminal and color the output then. But I don't know if there is a way to do this. Also, making the stacktrace structured might facilitate doing the coloring after the fact. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
