Open fjetter opened 4 months ago
I likely have to go through a couple of edge cases and adjust some tests for this to work. I haven't tried how the silence_logs kwarg factors in but overall I think this change is a strictly positive improvement
_See test report for an extended history of previous test failures. This is useful for diagnosing flaky tests._
29 files ± 0 29 suites ±0 10h 57m 8s :stopwatch: + 1h 14m 12s 4 051 tests - 5 3 951 :white_check_mark: + 9 97 :zzz: - 9 3 :x: - 1 55 799 runs +7 577 53 634 :white_check_mark: +7 347 2 161 :zzz: +248 4 :x: - 1
For more details on these failures, see this check.
Results for commit 22608665. ± Comparison against base commit e4a05450.
:recycle: This comment has been updated with latest results.
I checked the behavior again. If silence_logs=True
we're still setting all the loggers to silent but with silent_logs=False
it is respecting the config set in the dask.config ctx manager. I think this behavior makes sense.
Initializing logging on import has many unwanted side-effects.
Most importantly, it is very difficult to configure / overwrite anything unless the config file is overwritten directly. This should delay log configuration until it is needed. This allows code like this to work
which just prints