Ethanfel/Comfyui-Nodes-Stats
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Remove redundant get_all_packages() call in get_package_stats

Browse Source 
node_counts already contains all packages from both mapper.mapping
and get_all_packages(), so the second call for installed_packages
was unnecessary duplicate work.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This commit is contained in:
Ethanfel
2026-02-22 14:05:47 +01:00
parent 161da60537
commit 371cbb2bed
1 changed files with 2 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
tracker.py
View File

@@ -136,7 +136,8 @@ class UsageTracker:
packages[pkg]["total_nodes"] = total packages[pkg]["total_nodes"] = total
# Packages only in DB (not in mapper) are uninstalled/disabled # Packages only in DB (not in mapper) are uninstalled/disabled
installed_packages = set(node_counts.keys()) | mapper.get_all_packages() # node_counts already includes all packages from mapper + get_all_packages()
installed_packages = set(node_counts.keys())
for pkg, entry in packages.items(): for pkg, entry in packages.items():
if "total_nodes" not in entry: if "total_nodes" not in entry:
entry["total_nodes"] = entry["used_nodes"] entry["total_nodes"] = entry["used_nodes"]