STYLE: Replaced print calls to logging calls

fedorov / MultiVolumeImporter

A 3D Slicer module to support import of multi-volume data from non-DICOM sources, and provide plugin for importing such data from images saved in DICOM.

http://wiki.slicer.org/slicerWiki/index.php/Documentation/Nightly/Modules/MultiVolumeImporter

Other

9 stars 25 forks source link

STYLE: Replaced print calls to logging calls #13

Closed cpinter closed 9 years ago

cpinter commented 9 years ago

Messages such as "MultiVolumeImportPlugin::examine DICOMMultiVolumePlugin found 0 multivolumes!" appear every time when loading any DICOM series, adding unnecessary entries in the console output. These calls are replaced with logging.debug, so they do not appear in the console, and in case of error, logging.error is called.

fedorov commented 9 years ago

Sorry about this Csaba, and thank you for the fix! I've been annoyed myself about this, but didn't get around to fix it... Sorry.

cpinter commented 9 years ago

No problem, Andrey, I just had some idle minutes that's all :) One note though that I haven't tested it, as it's a very trivial change. If you see something that's not trivial in it, then I can run some tests.

fedorov commented 9 years ago

I agree, it all makes sense. I fixed another issue that I've been procrastinating, and committed hash update in Slicer r24664. Thanks!

cpinter commented 9 years ago

Thank you!