Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Don't strip ANSI from stdout (fixes #2365) #2366

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Don't strip ANSI from stdout (fixes #2365) #2366

wants to merge 2 commits into from

Commits on Oct 21, 2024

  1. Remove ANSI character stripping from STDOUT upon colorama init 

    - Until now ANSI was stripped from stdout if bittensor was not running within a TTY (like in PM2)
- This caused colored output to be broken under PM2 (and other process managers which do not give a TTY do bittensor/python
- Bittensor posseses its own file logger, which makes this behavior unnecessary and unwanted.
- This patch disables that behavior.
    @vaqxai
    vaqxai authored Oct 21, 2024
    Configuration menu
    Copy the full SHA
    6775e37 View commit details
    Browse the repository at this point in the history

  2. Change colorama init to not wrap output 

    Only without output wrapping does terminal output under PM2 behave correctly.
    @vaqxai
    vaqxai authored Oct 21, 2024
    Configuration menu
    Copy the full SHA
    028b09a View commit details
    Browse the repository at this point in the history