commontk / CTK

A set of common support code for medical imaging, surgical navigation, and related purposes.
https://commontk.org
Apache License 2.0
827 stars 481 forks source link

ENH: Remove unnecessary logging information from ctkDICOMDatabase class #1187

Closed Punzo closed 5 months ago

Punzo commented 5 months ago

Fix https://github.com/commontk/CTK/issues/1186

Punzo commented 5 months ago

@lassoan @jcfr

Punzo commented 5 months ago

I would make some adjustments (see comments inline) but overall I like this simple solution: if something is important for user to know or useful for troubleshooting (and does not happen to frequently) then log the message, otherwise remove the log.

ok and thanks for the review. I have applied it! @lassoan