Update app.py

app.py

@@ -287,15 +287,20 @@ def get_trending_accounts_with_commits(limit=100):
         progress_text.empty()
         progress_bar.empty()
         
-        # Sort by score (combination of commits and repo counts)
+        # Sort by score (commits only) for overall ranking
         user_stats.sort(key=lambda x: x["score"], reverse=True)
         
         # Extract rankings
         trending_authors = [stat["username"] for stat in user_stats[:limit]]
         
-        # Create detailed rankings for display
-        spaces_rank_data = [(stat["username"], stat["spaces"]) for stat in user_stats if stat["spaces"] > 0][:limit]
-        models_rank_data = [(stat["username"], stat["models"]) for stat in user_stats if stat["models"] > 0][:limit]
+        # Create separate rankings sorted by spaces and models count
+        spaces_rank_data = [(stat["username"], stat["spaces"]) for stat in user_stats if stat["spaces"] > 0]
+        spaces_rank_data.sort(key=lambda x: x[1], reverse=True)  # Sort by spaces count
+        spaces_rank_data = spaces_rank_data[:limit]
+        
+        models_rank_data = [(stat["username"], stat["models"]) for stat in user_stats if stat["models"] > 0]
+        models_rank_data.sort(key=lambda x: x[1], reverse=True)  # Sort by models count
+        models_rank_data = models_rank_data[:limit]
         
         return trending_authors, spaces_rank_data, models_rank_data, user_stats[:limit]
         
@@ -824,8 +829,8 @@ with st.sidebar:
             )
 
     with tab2:
-        # Show trending accounts by Spaces & Models
-        st.markdown('<div class="subheader"><h3>🚀 Spaces Leaders</h3></div>', unsafe_allow_html=True)
+        # Show accounts sorted by Spaces count
+        st.markdown('<div class="subheader"><h3>🚀 Top 50 by Spaces Count</h3></div>', unsafe_allow_html=True)
         
         # Create a data frame for the Spaces table with medals for top 3
         if top_owners_spaces:
@@ -855,8 +860,8 @@ with st.sidebar:
                 hide_index=False
             )
         
-        # Display the top Models accounts list with medals for top 3
-        st.markdown('<div class="subheader"><h3>🧠 Models Leaders</h3></div>', unsafe_allow_html=True)
+        # Display accounts sorted by Models count
+        st.markdown('<div class="subheader"><h3>🧠 Top 50 by Models Count</h3></div>', unsafe_allow_html=True)
         
         # Create a data frame for the Models table with medals for top 3
         if top_owners_models:
@@ -934,11 +939,11 @@ if username:
     # Create a header card with contributor info
     header_col1, header_col2 = st.columns([1, 2])
     with header_col1:
-        score_display = f"Score: {user_stat['score']:.1f}" if user_stat else "Score: N/A"
+        commits_display = f"Est. Commits: {user_stat['estimated_commits']}" if user_stat else "Est. Commits: N/A"
         st.markdown(f'<div style="background-color: #E3F2FD; padding: 20px; border-radius: 10px; border-left: 5px solid #1E88E5;">'
                   f'<h2 style="color: #1E88E5;">👤 {username}</h2>'
                   f'<p style="font-size: 16px;">Analyzing contributions for {selected_year}</p>'
-                  f'<p style="font-size: 14px; font-weight: bold;">{score_display}</p>'
+                  f'<p style="font-size: 14px; font-weight: bold;">{commits_display}</p>'
                   f'<p><a href="https://huggingface.co/{username}" target="_blank" style="color: #1E88E5; font-weight: bold;">View Profile</a></p>'
                   f'</div>', unsafe_allow_html=True)
     
@@ -947,7 +952,7 @@ if username:
         st.markdown(f'<div style="background-color: #F3E5F5; padding: 20px; border-radius: 10px; border-left: 5px solid #9C27B0;">'
                   f'<h3 style="color: #9C27B0;">About This Analysis</h3>'
                   f'<p>This dashboard analyzes {username}\'s contributions to Hugging Face in {selected_year}, including models, datasets, and spaces.</p>'
-                  f'<p style="font-style: italic; font-size: 12px;">* Rankings are based on contribution scores combining repos and commit activity.</p>'
+                  f'<p style="font-style: italic; font-size: 12px;">* Overall rankings are based on total commit count. Space/Model rankings are based on repository count.</p>'
                   f'</div>', unsafe_allow_html=True)
 
     with st.spinner(f"Fetching detailed contribution data for {username}..."):