You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
It is perfectly fine to have the rich mark up when I have rich installed, but I also want to listen to the Kedro log handler so I can export these logs to external application.
The mark up syntax are recorded as part of the log and there doesn't seem to be a way to remove it completely. The only thing that I can do is to remove rich completely. Ideally I can keep rich at the terminal/client, but having a way to export the log message without the rich mark up.
The text was updated successfully, but these errors were encountered:
@astrojuanlu It's possible to remove rich logger, but that's not what I want to do. I want to keep Kedro default as is, but having my own log handler to do application specific work (i.e. log to mlflow). I will try to work out a PR today, if not I can at least provide a test case so you can quickly validate this.
At the moment the root cause is right now the log is done at DataCatalog level:
So as long as one of the LogHandler is RichHandler, all handler will receive the markup log. My proposal is to move that markup logic to kedro.logging.RichHandler
Description
Terminal:

Application:

Context
It is perfectly fine to have the rich mark up when I have rich installed, but I also want to listen to the Kedro log handler so I can export these logs to external application.
The mark up syntax are recorded as part of the log and there doesn't seem to be a way to remove it completely. The only thing that I can do is to remove
rich
completely. Ideally I can keeprich
at the terminal/client, but having a way to export the log message without the rich mark up.The text was updated successfully, but these errors were encountered: