search

kirbyju / tcia_utils

A package to simplify common tasks one might perform when interacting with The Cancer Imaging Archive (TCIA) via Jupyter/Python.
Apache License 2.0
14 stars 4 forks source link

Logging instead of print statements? #1

Closed zjp closed 1 year ago

zjp commented 1 year ago

Hi, I'm currently using this package to power a tool in UCSF ChimeraX but I'm finding that it prints a lot of information to our log, which captures print statements and redirects them to itself.

It would be nice to go through the logging module instead, so that I (and other consumers) could do something like

import logging
logging.getLogger('tcia_utils').setLevel(100)

to silence output messages, as we do with other libraries we use.

I am 100% willing to write the PR if you're amenable to the change, I just thought starting with an issue would be less presumptuous than a PR out of nowhere.

kirbyju commented 1 year ago

That sounds like an excellent contribution. Thanks for offering to help!