Skip to content
Snippets Groups Projects
Commit c61c473d authored by Pierre-Yves David's avatar Pierre-Yves David :octopus:
Browse files

benchrepo: phase out the usage of "repo-hash"

The fact we use a hash of anything is an implementation details. In most case,
directly using "repo-prefix" is what one should do. We implement that in the
parsing code.
parent 3e5e26e0
No related branches found
No related tags found
No related merge requests found
......@@ -106,7 +106,7 @@
repo_ids = set()
format_variants = collections.defaultdict(set)
repo_details = {}
repo_hash_map = {}
repo_map = {}
partial_sets_set = set()
# Parse the benchrepos files
......@@ -117,7 +117,6 @@
repo_id = reference_repo["id"]
repo_ids.add(repo_id)
repo_hash = reference_repo.get("repo-hash", None)
format_info = reference_repo.get("format-info", {})
partial_sets = reference_repo.get("partial-sets", {})
......@@ -133,9 +132,8 @@
repo_name = reference_repo['repo-prefix']
repo_details[repo_name] = data
if repo_hash is not None:
repo_hash_map_key = (repo_id, tuple(sorted(new_format_info.items())))
repo_hash_map[repo_hash_map_key] = repo_name
repo_map_key = (repo_id, tuple(sorted(new_format_info.items())))
repo_map[repo_map_key] = repo_name
params = [list(sorted(repo_ids))]
params_names = ["repo"]
......@@ -149,7 +147,7 @@
params_names,
len(params_names),
repo_details,
repo_hash_map,
repo_map,
list(sorted(partial_sets_set)),
)
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment