hockey-blast-common-lib 0.1.57__tar.gz → 0.1.59__tar.gz

{hockey_blast_common_lib-0.1.57 → hockey_blast_common_lib-0.1.59}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: hockey-blast-common-lib
-Version: 0.1.57
+Version: 0.1.59
 Summary: Common library for shared functionality and DB models
 Author: Pavel Kletskov
 Author-email: kletskov@gmail.com

{hockey_blast_common_lib-0.1.57 → hockey_blast_common_lib-0.1.59}/hockey_blast_common_lib/aggregate_scorekeeper_stats.py

@@ -52,13 +52,14 @@ def calculate_quality_score(avg_max_saves_5sec, avg_max_saves_20sec, peak_max_sa
     return round(score, 2)
 
 def aggregate_scorekeeper_stats(session, aggregation_type, aggregation_id, names_to_filter_out, aggregation_window=None):
+    # Only process scorekeeper stats for ALL_ORGS_ID - skip individual organizations
+    # This prevents redundant processing when upstream logic calls with all organization IDs
+    if aggregation_type == 'org' and aggregation_id != ALL_ORGS_ID:
+        return  # Do nothing for individual organization IDs
+
     human_ids_to_filter = get_human_ids_by_names(session, names_to_filter_out)
 
     if aggregation_type == 'org':
-        # Only process scorekeeper stats for ALL_ORGS_ID - skip individual organizations
-        if aggregation_id != ALL_ORGS_ID:
-            return  # Skip processing for individual organization IDs
-
         aggregation_name = "All Orgs"
         filter_condition = sqlalchemy.true()  # No filter for organization
         print(f"Aggregating scorekeeper stats for {aggregation_name} with window {aggregation_window}...")
@@ -162,7 +163,6 @@ def aggregate_scorekeeper_stats(session, aggregation_type, aggregation_id, names
     assign_ranks(stats_dict, 'quality_score', reverse_rank=True)  # Lower is better (less problematic)
 
     # Insert aggregated stats into the appropriate table with progress output
-    total_items = len(stats_dict)
     batch_size = 1000
     for i, (key, stat) in enumerate(stats_dict.items(), 1):
         aggregation_id, human_id = key
@@ -203,20 +203,29 @@ def run_aggregate_scorekeeper_stats():
     session = create_session("boss")
     human_id_to_debug = None
 
-    # Process ALL_ORGS scorekeeper stats only (cross-organizational)
-    # Scorekeeper quality should be measured across all organizations they work for
-    if human_id_to_debug is None:
-        org_progress = create_progress_tracker(3, f"Processing cross-organizational scorekeeper stats")
-        aggregate_scorekeeper_stats(session, aggregation_type='org', aggregation_id=ALL_ORGS_ID, names_to_filter_out=not_human_names)
-        org_progress.update(1)
-        aggregate_scorekeeper_stats(session, aggregation_type='org', aggregation_id=ALL_ORGS_ID, names_to_filter_out=not_human_names, aggregation_window='Weekly')
-        org_progress.update(2)
-        aggregate_scorekeeper_stats(session, aggregation_type='org', aggregation_id=ALL_ORGS_ID, names_to_filter_out=not_human_names, aggregation_window='Daily')
-        org_progress.update(3)
-    else:
-        aggregate_scorekeeper_stats(session, aggregation_type='org', aggregation_id=ALL_ORGS_ID, names_to_filter_out=not_human_names)
-        aggregate_scorekeeper_stats(session, aggregation_type='org', aggregation_id=ALL_ORGS_ID, names_to_filter_out=not_human_names, aggregation_window='Weekly')
-        aggregate_scorekeeper_stats(session, aggregation_type='org', aggregation_id=ALL_ORGS_ID, names_to_filter_out=not_human_names, aggregation_window='Daily')
+    # Get all org_id present in the Organization table (following goalie stats pattern)
+    # Individual org calls will be skipped by early exit, only ALL_ORGS_ID will process
+    from hockey_blast_common_lib.models import Organization
+    org_ids = session.query(Organization.id).all()
+    org_ids = [org_id[0] for org_id in org_ids]
+
+    # Add ALL_ORGS_ID to the list so it gets processed
+    org_ids.append(ALL_ORGS_ID)
+
+    for org_id in org_ids:
+        if human_id_to_debug is None:
+            org_name = "All Organizations" if org_id == ALL_ORGS_ID else session.query(Organization.organization_name).filter(Organization.id == org_id).scalar() or f"org_id {org_id}"
+            org_progress = create_progress_tracker(3, f"Processing scorekeeper stats for {org_name}")
+            aggregate_scorekeeper_stats(session, aggregation_type='org', aggregation_id=org_id, names_to_filter_out=not_human_names)
+            org_progress.update(1)
+            aggregate_scorekeeper_stats(session, aggregation_type='org', aggregation_id=org_id, names_to_filter_out=not_human_names, aggregation_window='Weekly')
+            org_progress.update(2)
+            aggregate_scorekeeper_stats(session, aggregation_type='org', aggregation_id=org_id, names_to_filter_out=not_human_names, aggregation_window='Daily')
+            org_progress.update(3)
+        else:
+            aggregate_scorekeeper_stats(session, aggregation_type='org', aggregation_id=org_id, names_to_filter_out=not_human_names)
+            aggregate_scorekeeper_stats(session, aggregation_type='org', aggregation_id=org_id, names_to_filter_out=not_human_names, aggregation_window='Weekly')
+            aggregate_scorekeeper_stats(session, aggregation_type='org', aggregation_id=org_id, names_to_filter_out=not_human_names, aggregation_window='Daily')
 
 if __name__ == "__main__":
     run_aggregate_scorekeeper_stats()

{hockey_blast_common_lib-0.1.57 → hockey_blast_common_lib-0.1.59}/hockey_blast_common_lib.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: hockey-blast-common-lib
-Version: 0.1.57
+Version: 0.1.59
 Summary: Common library for shared functionality and DB models
 Author: Pavel Kletskov
 Author-email: kletskov@gmail.com

{hockey_blast_common_lib-0.1.57 → hockey_blast_common_lib-0.1.59}/setup.py

@@ -2,7 +2,7 @@ from setuptools import setup, find_packages
 
 setup(
     name='hockey-blast-common-lib',  # The name of your package
-    version='0.1.57',
+    version='0.1.59',
     description='Common library for shared functionality and DB models',
     author='Pavel Kletskov',
     author_email='kletskov@gmail.com',