acoustic-carpenter-78188
06/15/2023, 1:51 PMflytekit==1.6.0 rich tracebacks were introduced (#1582).
Before, exceptions were shown like this:
Screenshot 2023-06-15 at 15 37 04ā¾
Screenshot 2023-06-15 at 15 36 44ā¾
import flytekit is called somewhere in the imported code base, also when not directly working with flyte.
* * *
Style ultimately is a question of taste but several of our engineers have complained about this change:
When/Why did the stacktrace get souglycolorful. It takes up way too much space and pycharm cannot create crosslinks to the files anymore for me š And long lines are truncated
I feel the same, does not help me
Can we configure this
I wanted to complain about this too š
The tracebacks contain way more (white space) characters, meaning I can paste a lot less of them into chat gptBecause of this, in this PR I propose to introduce an environment variable which disables rich tracebacks. I chose a naming inspired by
"FLYTE_SDK_LOGGING_LEVEL" or "FLYTE_SDK_LOGGING_FORMAT" in flytekit/loggers.py.
Would this change be be acceptable to you @cosmicBboy @freezing-airport-6809 @eapolinario ?
Type
ā Bug Fix
āļø Feature
ā Plugin
Are all requirements met?
āļø Code completed
āļø Smoke tested
ā Unit tests added
ā Code documentation added
ā Any pending items have an associated Issue
Complete description
NA
Tracking Issue
NA
Follow-up issue
NA
flyteorg/flytekit
ā
All checks have passed
30/30 successful checksacoustic-carpenter-78188
06/15/2023, 9:19 PM