treeverse/dvc

Pattern Findings

1460 matches across 10 categories. Click a row to expand file-level details.

Hyper-Verbose Identifiers863 hits · 875 pts

Severity	File	Line	Snippet
LOW	tests/unit/test_ignore.py	167	def test_match_ignore_from_file(
LOW	tests/unit/test_ignore.py	303	def test_ignore_unignore_from_git_example(tmp_dir, dvc, scm):
LOW	tests/unit/test_pathspec_math.py	71	def test_dvcignore_pattern_change_dir(tmp_dir, patterns, dirname, changed):
LOW	tests/unit/test_dirs.py	9	def test_global_config_dir_respects_env_var(monkeypatch):
LOW	tests/unit/test_dirs.py	16	def test_site_cache_dir_on_unix(monkeypatch):
LOW	tests/unit/test_dirs.py	21	def test_site_cache_dir_env_var(monkeypatch):
LOW	tests/unit/test_dirs.py	26	def test_site_cache_dir_with_config_parameter(monkeypatch):
LOW	tests/unit/test_dirs.py	31	def test_site_cache_dir_env_var_precedence(monkeypatch):
LOW	tests/unit/test_prompt.py	4	def test_confirm_in_tty_if_stdin_is_closed(mocker):
LOW	tests/unit/test_tabular_data.py	99	def test_dict_like_interfaces():
LOW	tests/unit/test_tabular_data.py	296	def test_drop_duplicates_rich_text():
LOW	tests/unit/test_tabular_data.py	349	def test_drop_duplicates_subset(axis, subset, expected):
LOW	tests/unit/test_tabular_data.py	374	def test_drop_duplicates_invalid_axis():
LOW	tests/unit/test_updater.py	71	def test_check_update_respect_config(result, updater, mocker):
LOW	tests/unit/test_updater.py	107	def test_check_refetches_each_day(mock_tty, updater, caplog, mocker):
LOW	tests/unit/test_updater.py	124	def test_check_fetches_on_invalid_data_format(mock_tty, updater, caplog, mocker):
LOW	tests/unit/test_lockfile.py	8	def test_stage_dump_no_outs_deps(tmp_dir, dvc):
LOW	tests/unit/test_lockfile.py	18	def test_stage_dump_when_already_exists(tmp_dir, dvc):
LOW	tests/unit/test_lockfile.py	30	def test_stage_dump_with_deps_and_outs(tmp_dir, dvc):
LOW	tests/unit/test_lockfile.py	48	def test_stage_overwrites_if_already_exists(tmp_dir, dvc):
LOW	tests/unit/test_lockfile.py	60	def test_load_when_lockfile_does_not_exist(tmp_dir, dvc):
LOW	tests/unit/test_lockfile.py	78	def test_load_when_lockfile_is_corrupted(tmp_dir, dvc, corrupt_data):
LOW	tests/unit/test_lockfile.py	88	def test_try_loading_lockfile_that_is_gitignored(
LOW	tests/unit/test_analytics.py	24	def test_collect_and_send_report(mocker, tmp_global_dir):
LOW	tests/unit/test_analytics.py	194	def test_git_remote_hash_local(mocker, git_remote):
LOW	tests/unit/test_progress.py	16	def test_quiet_logging_disable_false(caplog, capsys, mocker):
LOW	tests/unit/test_scm.py	7	def test_resolve_rev_empty_git_repo(scm):
LOW	tests/unit/test_dvcfile.py	34	def test_pipelines_single_stage_file(path):
LOW	tests/unit/test_dvcfile.py	41	def test_stage_load_on_not_existing_file(tmp_dir, dvc, file, is_dvcignored):
LOW	tests/unit/test_dvcfile.py	54	def test_stage_load_on_non_file(tmp_dir, dvc, file):
LOW	tests/unit/test_dvcfile.py	62	def test_stage_load_on_invalid_data(tmp_dir, dvc, file):
LOW	tests/unit/test_dvcfile.py	93	def test_dump_multiple_pipeline_stages(tmp_dir, dvc):
LOW	tests/unit/test_dvcfile.py	115	def test_dump_stages_single_stage(tmp_dir, dvc):
LOW	tests/unit/test_dvcfile.py	127	def test_stage_load_file_exists_but_dvcignored(tmp_dir, dvc, scm, file):
LOW	tests/unit/test_dvcfile.py	140	def test_try_loading_dvcfile_that_is_gitignored(tmp_dir, dvc, scm, file):
LOW	tests/unit/test_dvcfile.py	155	def test_dvcfile_encoding_error(tmp_dir, dvc):
LOW	tests/unit/test_rwlock.py	59	def test_rwlock_edit_is_guarded(tmp_path, mocker):
LOW	tests/unit/test_config.py	119	def test_feature_section_supports_arbitrary_values(caplog):
LOW	tests/unit/test_context.py	40	def test_context_dict_ignores_keys_except_str():
LOW	tests/unit/test_context.py	66	def test_context_setitem_getitem():
LOW	tests/unit/test_context.py	213	def test_overwrite_with_setitem():
LOW	tests/unit/test_context.py	322	def test_track_from_multiple_files(tmp_dir):
LOW	tests/unit/test_context.py	400	def test_resolve_resolves_dict_keys():
LOW	tests/unit/test_context.py	409	def test_resolve_resolves_boolean_value():
LOW	tests/unit/test_context.py	420	def test_load_from_raises_if_file_not_exist(tmp_dir, dvc):
LOW	tests/unit/test_context.py	427	def test_load_from_raises_if_file_is_directory(tmp_dir, dvc):
LOW	tests/unit/test_api.py	8	def test_open_raises_error_if_no_context(tmp_dir, dvc):
LOW	tests/unit/test_api.py	18	def test_open_rev_raises_error_on_wrong_mode(tmp_dir, dvc):
LOW	tests/unit/test_compare.py	26	def test_diff_table_with_value_column():
LOW	tests/unit/test_compare.py	69	def test_diff_table_precision():
LOW	tests/unit/test_compare.py	100	def test_diff_unsupported_diff_message(extra, expected):
LOW	tests/unit/test_compare.py	267	def test_metrics_show_with_valid_falsey_values():
LOW	tests/unit/test_compare.py	289	def test_metrics_show_with_no_revision():
LOW	tests/unit/test_compare.py	305	def test_metrics_show_with_non_dict_values():
LOW	tests/unit/test_compare.py	313	def test_metrics_show_with_multiple_revision():
LOW	tests/unit/test_compare.py	343	def test_metrics_show_with_one_revision_multiple_paths():
LOW	tests/unit/test_compare.py	373	def test_metrics_show_with_different_metrics_header():
LOW	tests/unit/test_compare.py	405	def test_metrics_show_precision():
LOW	tests/unit/test_compare.py	471	def test_metrics_show_default(capsys):
LOW	tests/unit/test_compare.py	494	def test_metrics_show_markdown(capsys):
803 more matches not shown…

Cross-Language Confusion153 hits · 848 pts

Severity	File	Line	Snippet
HIGH	tests/func/test_ignore.py	470	dvc.push()
HIGH	tests/func/test_gc.py	229	dvc.push()
HIGH	tests/func/test_gc.py	354	dvc.push(["file1", "dir1"])
HIGH	tests/func/test_gc.py	388	dvc.push(["file1", "dir1"], remote="other_remote")
HIGH	tests/func/test_gc.py	413	dvc.push()
HIGH	tests/func/test_gc.py	442	dvc.push()
HIGH	tests/func/test_run_cache.py	19	assert dvc.push(run_cache=True) == 2
HIGH	tests/func/test_run_cache.py	90	cmd="cp foo bar && cp foo goo",
HIGH	tests/func/test_run_cache.py	216	dvc.push(run_cache=True)
HIGH	tests/func/test_run_cache.py	247	dvc.push(run_cache=True)
HIGH	tests/func/test_data_status.py	415	assert dvc.push() == 4
HIGH	tests/func/test_data_status.py	439	dvc.push(remote="myremote")
HIGH	tests/func/test_data_status.py	518	dvc.push()
HIGH	tests/func/test_data_status.py	916	dvc.push()
HIGH	tests/func/test_data_status.py	1235	assert dvc.push() == 3
HIGH	tests/func/test_data_status.py	1264	dvc.push()
HIGH	tests/func/test_external_repo.py	53	erepo_dir.dvc.push()
HIGH	tests/func/test_external_repo.py	112	erepo_dir.dvc.push()
HIGH	tests/func/test_status.py	127	cmd="echo alice>alice && echo bob>bob",
HIGH	tests/func/test_virtual_directory.py	20	assert dvc.push() == 3
HIGH	tests/func/test_virtual_directory.py	33	assert dvc.push() == 2
HIGH	tests/func/test_virtual_directory.py	41	assert dvc.push() == 2
HIGH	tests/func/test_virtual_directory.py	63	assert dvc.push() == 5
HIGH	tests/func/test_virtual_directory.py	75	assert dvc.push() == 1
HIGH	tests/func/test_virtual_directory.py	85	assert dvc.push() == 1
HIGH	tests/func/test_virtual_directory.py	98	assert dvc.push() == 3
HIGH	tests/func/test_virtual_directory.py	110	assert dvc.push() == 3
HIGH	tests/func/test_virtual_directory.py	123	assert dvc.push() == 3
HIGH	tests/func/test_virtual_directory.py	134	assert dvc.push() == 2
HIGH	tests/func/test_virtual_directory.py	147	assert dvc.push() == 3
HIGH	tests/func/test_virtual_directory.py	158	assert not dvc.push()
HIGH	tests/func/test_virtual_directory.py	166	assert not dvc.push()
HIGH	tests/func/test_virtual_directory.py	180	assert dvc.push() == 22
HIGH	tests/func/test_virtual_directory.py	198	assert dvc.push() == 2
HIGH	tests/func/test_import.py	355	erepo_dir.dvc.push([os.path.join(os.fspath(erepo_dir), "dire*")], glob=True)
HIGH	tests/func/test_import.py	438	erepo_dir.dvc.push()
HIGH	tests/func/test_import.py	455	dvc.push()
HIGH	tests/func/test_import.py	602	erepo_dir.dvc.push()
HIGH	tests/func/test_get.py	233	erepo_dir.dvc.push()
HIGH	tests/func/test_get.py	260	dvc.push()
HIGH	tests/func/test_remote.py	211	dvc.push()
HIGH	tests/func/test_remote.py	220	dvc.push()
HIGH	tests/func/test_remote.py	255	dvc.push()
HIGH	tests/func/test_remote.py	285	dvc.push()
HIGH	tests/func/test_remote.py	419	dvc.push()
HIGH	tests/func/test_remote.py	438	dvc.push()
HIGH	tests/func/test_install.py	92	scm.gitpython.repo.git.push("origin", "master")
HIGH	tests/func/test_data_cloud.py	134	dvc.push()
HIGH	tests/func/test_data_cloud.py	437	assert dvc.push(["dvc.yaml:copy-ipsum-baz"]) == 1
HIGH	tests/func/test_data_cloud.py	441	assert dvc.push(["dvc.yaml"]) == 2
HIGH	tests/func/test_data_cloud.py	445	dvc.push(["dvc.yaml:StageThatDoesNotExist"])
HIGH	tests/func/test_data_cloud.py	531	assert dvc.push(**{key: True}) == expected
HIGH	tests/func/test_data_cloud.py	544	assert dvc.push("copy-foo-bar") == 1
HIGH	tests/func/test_data_cloud.py	548	assert dvc.push() == 0
HIGH	tests/func/test_data_cloud.py	177	dvc.push()
HIGH	tests/func/test_data_cloud.py	198	dvc.push()
HIGH	tests/func/test_data_cloud.py	332	dvc.push("bar")
HIGH	tests/func/test_data_cloud.py	363	dvc.push()
HIGH	tests/func/test_data_cloud.py	414	assert dvc.push() == 3
HIGH	tests/func/test_data_cloud.py	477	dvc.push()
93 more matches not shown…

Unused Imports309 hits · 305 pts

Severity	File	Line
LOW	tests/conftest.py	10
LOW	tests/conftest.py	12
LOW	tests/conftest.py	13
LOW	tests/conftest.py	14
LOW	tests/integration/conftest.py	1
LOW	tests/integration/conftest.py	1
LOW	tests/utils/asserts.py	5
LOW	tests/remotes/__init__.py	1
LOW	tests/remotes/__init__.py	1
LOW	tests/remotes/__init__.py	1
LOW	tests/remotes/__init__.py	3
LOW	tests/remotes/__init__.py	3
LOW	tests/func/test_dataset.py	26
LOW	tests/func/test_data_cloud.py	15
LOW	tests/func/experiments/conftest.py	3
LOW	tests/func/experiments/conftest.py	3
LOW	tests/func/experiments/conftest.py	3
LOW	tests/func/experiments/conftest.py	3
LOW	tests/func/experiments/conftest.py	3
LOW	tests/func/experiments/conftest.py	3
LOW	tests/func/api/test_data.py	9
LOW	tests/func/api/test_experiments.py	7
LOW	dvc/config.py	19
LOW	dvc/config.py	20
LOW	dvc/data_cloud.py	13
LOW	dvc/data_cloud.py	14
LOW	dvc/data_cloud.py	15
LOW	dvc/config_schema.py	23
LOW	dvc/database.py	19
LOW	dvc/database.py	19
LOW	dvc/database.py	19
LOW	dvc/database.py	20
LOW	dvc/__init__.py	8
LOW	dvc/__init__.py	9
LOW	dvc/__init__.py	9
LOW	dvc/types.py	4
LOW	dvc/updater.py	12
LOW	dvc/ignore.py	25
LOW	dvc/ignore.py	27
LOW	dvc/dvcfile.py	19
LOW	dvc/dvcfile.py	20
LOW	dvc/dvcfile.py	23
LOW	dvc/cachemgr.py	9
LOW	dvc/exceptions.py	9
LOW	dvc/analytics.py	10
LOW	dvc/scm.py	10
LOW	dvc/scm.py	19
LOW	dvc/scm.py	21
LOW	dvc/_debug.py	6
LOW	dvc/_debug.py	7
LOW	dvc/output.py	47
LOW	dvc/output.py	49
LOW	dvc/ui/__init__.py	10
LOW	dvc/ui/__init__.py	11
LOW	dvc/ui/__init__.py	11
LOW	dvc/ui/__init__.py	12
LOW	dvc/ui/__init__.py	13
LOW	dvc/ui/__init__.py	14
LOW	dvc/ui/__init__.py	17
LOW	dvc/ui/__init__.py	18
249 more matches not shown…

Deep Nesting72 hits · 72 pts

Severity	File	Line
LOW	tests/unit/command/test_help.py	13
LOW	tests/unit/command/test_help.py	17
LOW	tests/unit/repo/experiments/queue/test_celery.py	176
LOW	tests/unit/repo/experiments/queue/test_celery.py	190
LOW	dvc/rwlock.py	76
LOW	dvc/logger.py	143
LOW	dvc/dagascii.py	83
LOW	dvc/lock.py	110
LOW	dvc/dvcfile.py	427
LOW	dvc/cachemgr.py	41
LOW	dvc/_debug.py	34
LOW	dvc/compare.py	182
LOW	dvc/compare.py	226
LOW	dvc/output.py	425
LOW	dvc/output.py	912
LOW	dvc/render/match.py	68
LOW	dvc/render/converter/vega.py	22
LOW	dvc/render/converter/vega.py	33
LOW	dvc/render/converter/vega.py	85
LOW	dvc/render/converter/vega.py	122
LOW	dvc/dependency/param.py	124
LOW	dvc/utils/collections.py	6
LOW	dvc/utils/table.py	14
LOW	dvc/utils/table.py	48
LOW	dvc/utils/serialize/_py.py	88
LOW	dvc/utils/serialize/_py.py	114
LOW	dvc/cli/__init__.py	151
LOW	dvc/testing/benchmarks/fixtures.py	95
LOW	dvc/testing/benchmarks/cli/stories/test_modify_data.py	38
LOW	dvc/api/experiments.py	46
LOW	dvc/api/data.py	258
LOW	dvc/stage/__init__.py	313
LOW	dvc/stage/__init__.py	420
LOW	dvc/commands/add.py	10
LOW	dvc/commands/gc.py	13
LOW	dvc/commands/remote.py	82
LOW	dvc/commands/plots.py	75
LOW	dvc/commands/plots.py	184
LOW	dvc/commands/diff.py	118
LOW	dvc/commands/experiments/ls.py	12
LOW	dvc/commands/experiments/diff.py	13
LOW	dvc/parsing/interpolate.py	96
LOW	dvc/fs/dvc.py	378
LOW	dvc/fs/dvc.py	447
LOW	dvc/repo/gc.py	55
LOW	dvc/repo/index.py	47
LOW	dvc/repo/index.py	946
LOW	dvc/repo/index.py	467
LOW	dvc/repo/index.py	653
LOW	dvc/repo/commit.py	46
LOW	dvc/repo/__init__.py	102
LOW	dvc/repo/open_repo.py	155
LOW	dvc/repo/checkout.py	42
LOW	dvc/repo/diff.py	25
LOW	dvc/repo/worktree.py	305
LOW	dvc/repo/data.py	408
LOW	dvc/repo/experiments/collect.py	126
LOW	dvc/repo/experiments/run.py	15
LOW	dvc/repo/experiments/pull.py	22
LOW	dvc/repo/experiments/__init__.py	138
12 more matches not shown…

Excessive Try-Catch Wrapping39 hits · 42 pts

Severity	File	Line	Snippet
LOW	tests/unit/test_logger.py	65	except Exception:
LOW	tests/unit/test_logger.py	76	except Exception:
LOW	tests/unit/test_logger.py	87	except Exception:
LOW	tests/unit/test_logger.py	98	except Exception:
LOW	tests/unit/test_logger.py	116	except Exception:
LOW	tests/unit/test_logger.py	136	except Exception:
LOW	tests/unit/test_logger.py	154	except Exception as exc: # noqa: BLE001
LOW	dvc/database.py	84	except Exception as exc:
LOW	dvc/logger.py	153	except Exception: # noqa: BLE001, S110
LOW	dvc/logger.py	161	except Exception: # noqa: BLE001
LOW	dvc/updater.py	80	except Exception as e: # noqa: BLE001
LOW	dvc/analytics.py	150	except Exception:
LOW	dvc/analytics.py	178	except Exception as exc: # noqa: BLE001
LOW	dvc/_debug.py	124	except Exception:
MEDIUM	dvc/_debug.py	121	def debug():
LOW	dvc/render/match.py	114	except Exception as e: # noqa: BLE001
LOW	dvc/utils/__init__.py	393	except Exception as e: # noqa: BLE001
LOW	dvc/cli/__init__.py	236	except Exception as exc: # noqa: BLE001
LOW	dvc/repo/fetch.py	88	except Exception as exc: # noqa: BLE001
LOW	dvc/repo/reproduce.py	188	except Exception as exc: # noqa: BLE001
LOW	dvc/repo/scm_context.py	104	except Exception:
LOW	dvc/repo/artifacts.py	268	except Exception as exc:
LOW	dvc/repo/artifacts.py	336	except Exception as exc: # noqa: BLE001
LOW	dvc/repo/artifacts.py	364	except Exception as exc: # noqa: BLE001
LOW	dvc/repo/artifacts.py	378	except Exception as exc:
LOW	dvc/repo/metrics/show.py	69	except Exception as exc: # noqa: BLE001
LOW	dvc/repo/metrics/show.py	126	except Exception as e:
LOW	dvc/repo/metrics/show.py	227	except Exception as exc:
LOW	dvc/repo/experiments/collect.py	61	except Exception as exc:
LOW	dvc/repo/experiments/stash.py	144	except Exception:
LOW	dvc/repo/experiments/push.py	127	except Exception as exc: # noqa: BLE001
LOW	dvc/repo/experiments/queue/tempdir.py	128	except Exception as exc:
LOW	dvc/repo/experiments/queue/tasks.py	75	except Exception:
LOW	dvc/repo/experiments/queue/workspace.py	134	except Exception as exc:
LOW	dvc/repo/experiments/queue/base.py	427	except Exception: # noqa: BLE001
LOW	dvc/repo/experiments/executor/local.py	191	except Exception:
LOW	dvc/repo/experiments/executor/base.py	645	except Exception:
LOW	dvc/repo/params/show.py	111	except Exception as exc: # noqa: BLE001
LOW	dvc/repo/params/show.py	194	except Exception as exc:

Docstring Block Structure7 hits · 35 pts

Severity	File	Line	Snippet
HIGH	dvc/api/show.py	41	Get metrics tracked in `repo`. Without arguments, this function will retrieve all metrics from all tracked metr
HIGH	dvc/api/show.py	164	Get parameters tracked in `repo`. Without arguments, this function will retrieve all params from all tracked pa
HIGH	dvc/api/experiments.py	14	Create a new DVC experiment using `exp save`. See https://dvc.org/doc/command-reference/exp/save. Args:
HIGH	dvc/api/artifacts.py	13	Return path and Git revision for an artifact in a DVC project. The resulting path and revision can be used in
HIGH	dvc/api/data.py	112	Opens a file tracked in a DVC project. This function may only be used as a context manager (using the `with`
HIGH	dvc/repo/ls.py	57	Methods for getting files and outputs for the repo. Args: url (str): the repo url path (str, option
HIGH	dvc/repo/init.py	16	Creates an empty repo on the given directory -- basically a `.dvc` directory with subdirectories for configurat

Hallucination Indicators1 hit · 10 pts

Severity	File	Line	Snippet
CRITICAL	dvc/commands/experiments/queue_worker.py	11	self.repo.experiments.celery_queue.worker.start(

Fake / Example Data6 hits · 8 pts

Severity	File	Line	Snippet
LOW	tests/unit/utils/test_humanize.py	43	text = "lorem ipsum"
LOW	tests/unit/utils/test_humanize.py	61	text = "lorem ipsum"
LOW	tests/unit/repo/test_scm_context.py	107	context.track_file("lorem ipsum")
LOW	tests/unit/repo/test_scm_context.py	108	assert context.files_to_track == {"foo", "lorem ipsum"}
LOW	tests/unit/repo/test_scm_context.py	116	assert set(match.groups()) == {"'lorem ipsum'", "foo"}
LOW	tests/func/test_virtual_directory.py	102	tmp_dir.gen({"dir": {"subdir": {"ipsum": "lorem ipsum", "file": "file"}}})

Over-Commented Block7 hits · 7 pts

Severity	File	Line	Snippet
LOW	tests/unit/test_ignore.py	21	("#to_ignore", ["\\#to_ignore"], True),
LOW	tests/unit/test_pathspec_math.py	21	# Put a backslash ("\") in front of the first "!" for patterns that
LOW	tests/unit/test_pathspec_math.py	41	# The range notation, e.g. [a-zA-Z], can be used to match one of the
LOW	dvc/utils/__init__.py	81	env[lp_key] = lp_orig
LOW	dvc/cli/utils.py	41	# see: https://bugs.python.org/issue22848
LOW	dvc/stage/run.py	81	kwargs = {"cwd": stage.wdir, "env": fix_env(None), "close_fds": True}
LOW	dvc/fs/__init__.py	121	#

Self-Referential Comments3 hits · 7 pts

Severity	File	Line	Snippet
MEDIUM	tests/unit/fs/test_dvcfs.py	719	# Create the test dirs
MEDIUM	tests/unit/fs/test_dvcfs.py	748	# Create the test dir
MEDIUM	dvc/cli/__init__.py	1	"""This module provides an entrypoint to the dvc cli and parsing utils."""

Score History

Severity Breakdown

Pattern Findings