voxel51 / eta

ETA: Extensible Toolkit for Analytics
https://voxel51.com
Apache License 2.0
29 stars 13 forks source link

Disable default logging modifications #567

Closed brimoor closed 2 years ago

brimoor commented 2 years ago

Currently, import eta triggers eta.core.logging.basic_setup() to be called, which modifies the user's root logger. This is bad manners for anyone who is simply using ETA as a dependency and not for it's module or pipeline constructs.

This PR removes the default logging configuration so that import eta by itself not cause any modifications to the root logger (it is only configured when ETA modules/pipelines are actually run).