We need to stop printing things to stdout, especially on initialization/GUI startup. Warnings should be used for warnings or as a last resort stderr should be used instead of stdout. Additionally, all on-import things in uwsift/__init__.py should really be moved to nearer to the argument parsing (GUI creation/startup). Similarly, access to config items should be done at use time not import time. These things are not fixed as part of this PR but FIXMEs were added.
CC @strandgren @ameraner
For the record, I needed to remove these print messages because it was stopping me from doing bash things like version=$(python -c "import uwsift; print(uwsift.__version__)") which is used in the conda-pack build scripts.
We need to stop printing things to stdout, especially on initialization/GUI startup. Warnings should be used for warnings or as a last resort stderr should be used instead of stdout. Additionally, all on-import things in
uwsift/__init__.pyshould really be moved to nearer to the argument parsing (GUI creation/startup). Similarly, access to config items should be done at use time not import time. These things are not fixed as part of this PR but FIXMEs were added.CC @strandgren @ameraner
For the record, I needed to remove these print messages because it was stopping me from doing bash things like
version=$(python -c "import uwsift; print(uwsift.__version__)")which is used in the conda-pack build scripts.