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.

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

logging #193

Open
davidegraff opened this issue Jan 23, 2023 · 1 comment
Open

logging #193

davidegraff opened this issue Jan 23, 2023 · 1 comment
Assignees
Labels
enhancement New feature or request

Comments

@davidegraff
Copy link

Describe the problem
TDC currently prints a bunch of status messages to standard out. This presents a few problems for the client:

  • clutters my terminal
  • it's impossible to selectively turn them off
  • it's non-trivial to save them alongside other logging messages if I want to inspect them at a later date

Describe the solution you'd like
Move most logging statements from print_sys calls to logger.* calls

I.e., at the top of most modules:

import logging

logger = logging.getLogger(__name__)

# print_sys(mesg)
logger.*(mesg)

where * is the appropriate method depending on the severity of the message (debug, info, warn, error, critical)

Additional context
this page contains some good resources for addressing this

@kexinhuang12345
Copy link
Collaborator

Thank you for the great suggestion, David! We will work on it!

@kexinhuang12345 kexinhuang12345 self-assigned this Jan 24, 2023
@kexinhuang12345 kexinhuang12345 added the enhancement New feature or request label Jan 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants