From 95f7bd80644989fbb54728d4569cc3ebbf829643 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Saugat=20Pachhai=20=28=E0=A4=B8=E0=A5=8C=E0=A4=97=E0=A4=BE?= =?UTF-8?q?=E0=A4=A4=29?= Date: Sun, 14 Jun 2020 14:41:30 +0545 Subject: [PATCH] (bugfix) get/import: fix broken progress bar PR #3672 (6d8499e) extended `LocalRemote::_get_plans` to return one `checksums` too. As all of the args from `_get_plans` was passed down to `download()`, it recognized extra arg of checksum as `no_progress_bar` due to which it became True and stopped showing progress bar at all. Fix #3874 --- dvc/remote/local.py | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/dvc/remote/local.py b/dvc/remote/local.py index b778ff46c6..9e6b7b6a2e 100644 --- a/dvc/remote/local.py +++ b/dvc/remote/local.py @@ -626,8 +626,12 @@ def _process( func = pbar.wrap_fn(func) with ThreadPoolExecutor(max_workers=jobs) as executor: if download: - fails = sum(executor.map(func, *dir_plans)) - fails += sum(executor.map(func, *file_plans)) + from_infos, to_infos, names, _ = ( + d + f for d, f in zip(dir_plans, file_plans) + ) + fails = sum( + executor.map(func, from_infos, to_infos, names) + ) else: # for uploads, push files first, and any .dir files last