Skip to content

Commit

Permalink
fix: remove redundant deletion of full_assign directory in get_cluste…
Browse files Browse the repository at this point in the history
…rs function
  • Loading branch information
absternator committed Nov 21, 2024
1 parent 6dc261a commit 4396446
Showing 1 changed file with 2 additions and 3 deletions.
5 changes: 2 additions & 3 deletions beebop/assignClusters.py
Original file line number Diff line number Diff line change
Expand Up @@ -123,13 +123,12 @@ def get_clusters(
update_external_clusters_csv(previous_query_clustering_file, not_found_q_names, external_clusters_not_found)
# update external_clusters
external_clusters.update(external_clusters_not_found)
# delete full_assign directory as dont need anymore
shutil.rmtree(assign_full_dir)

save_external_to_poppunk_clusters(
queries_names, queries_clusters, external_clusters, p_hash, fs
)
# delete full_assign directory as dont need anymore
shutil.rmtree(assign_full_dir)

result = assign_clusters_to_result(external_clusters.items())
else:
result = assign_clusters_to_result(zip(queries_names, queries_clusters))
Expand Down

0 comments on commit 4396446

Please sign in to comment.