Update app.py

app.py

@@ -195,13 +195,12 @@ class PaperList:
 
 class PaperManager:
     """
-    Manages sorting, pagination, and search queries for the list of papers.
+    Manages sorting, pagination, and repository-based sorting for the list of papers.
     """
     def __init__(self, paper_list: PaperList, papers_per_page=30):
         self.paper_list = paper_list
         self.papers_per_page = papers_per_page
         self.sort_method = "hot"              # Default sort method
-        self.top_time_frame = "all time"      # Default time frame for "Top" sorting
         self.sort_papers()
         # 'current_page' and 'total_pages' are set in 'sort_papers()'
 
@@ -240,27 +239,6 @@ class PaperManager:
                 df_sorted = df
         elif self.sort_method == "new":
             df_sorted = df.sort_values(by='date', ascending=False)  # Sort by 'date'
-        elif self.sort_method == "top":
-            # Filter based on the selected time frame
-            now = datetime.datetime.now(timezone.utc)
-            if self.top_time_frame == "day":
-                time_threshold = now - datetime.timedelta(days=1)
-            elif self.top_time_frame == "week":
-                time_threshold = now - datetime.timedelta(weeks=1)
-            elif self.top_time_frame == "month":
-                time_threshold = now - datetime.timedelta(days=30)
-            elif self.top_time_frame == "year":
-                time_threshold = now - datetime.timedelta(days=365)
-            elif self.top_time_frame == "all time":
-                time_threshold = datetime.datetime.min.replace(tzinfo=timezone.utc)
-            else:
-                time_threshold = datetime.datetime.min.replace(tzinfo=timezone.utc)
-            
-            # Convert 'date' column to datetime
-            df_sorted = df.copy()
-            df_sorted['date_parsed'] = pd.to_datetime(df_sorted['date'], errors='coerce').dt.tz_localize(timezone.utc, ambiguous='NaT', nonexistent='NaT')
-            df_sorted = df_sorted[df_sorted['date_parsed'] >= time_threshold]
-            df_sorted = df_sorted.sort_values(by='upvotes', ascending=False).drop(columns=['date_parsed'])
         elif self.sort_method == "most_models":
             df_sorted = df.sort_values(by='models_count', ascending=False)
         elif self.sort_method == "most_datasets":
@@ -278,17 +256,13 @@ class PaperManager:
 
     def set_sort_method(self, method, time_frame=None):
         """
-        Sets the sort method ('hot', 'new', 'top', 'most_models', 'most_datasets', 'most_spaces') and re-sorts the papers.
-        If 'top' is selected, also sets the time frame.
+        Sets the sort method ('hot', 'new', 'most_models', 'most_datasets', 'most_spaces') and re-sorts the papers.
         """
-        valid_methods = ["hot", "new", "top", "most_models", "most_datasets", "most_spaces"]
+        valid_methods = ["hot", "new", "most_models", "most_datasets", "most_spaces"]
         if method not in valid_methods:
             method = "hot"
         logger.info(f"Setting sort method to: {method}")
         self.sort_method = method
-        if method == "top" and time_frame:
-            self.top_time_frame = time_frame.lower()
-            logger.info(f"Setting top time frame to: {self.top_time_frame}")
         self.sort_papers()
         return True  # Assume success
 
@@ -448,15 +422,18 @@ logger.info("Scheduler shutdown registered.")
 
 def change_sort_method_ui(method: str, time_frame: str = "all time") -> str:
     """
-    Changes the sort method and, if 'top' is selected, sets the time frame.
+    Changes the sort method and, if applicable, sets additional parameters.
     """
     logger.info(f"Changing sort method to: {method} with time frame: {time_frame}")
     if method.lower() in ["most_models", "most_datasets", "most_spaces"]:
         paper_manager.set_sort_method(method.lower())
-    elif method.lower() == "top":
-        paper_manager.set_sort_method(method.lower(), time_frame)
-    else:
+    elif method.lower() == "hot":
         paper_manager.set_sort_method(method.lower())
+    elif method.lower() == "new":
+        paper_manager.set_sort_method(method.lower())
+    else:
+        # Default to 'hot' if method is unrecognized
+        paper_manager.set_sort_method("hot")
     return paper_manager.get_current_page_papers()
 
 
@@ -625,28 +602,22 @@ with demo:
                 </tr>
             </table>
             """)
-        # Sort Options and Time Frame (conditionally visible)
+        # Sort Options (Removed "Top" and its timeframe)
         with gr.Row():
             sort_radio = gr.Radio(
-                choices=["Hot", "New", "Top", "Most Models", "Most Datasets", "Most Spaces"],
+                choices=["Hot", "New", "Most Models", "Most Datasets", "Most Spaces"],
                 value="Hot",
                 label="Sort By",
                 interactive=True
             )
-            time_frame_dropdown = gr.Dropdown(
-                choices=["day", "week", "month", "year", "all time"],
-                value="all time",
-                label="Time Frame for Top",
-                visible=False,
-                interactive=True
-            )
+            # Removed time_frame_dropdown as "Top" sort is removed
         # Paper list
         paper_list = gr.HTML()
         # Navigation Buttons
         with gr.Row():
             prev_button = gr.Button("Prev")
             next_button = gr.Button("Next")
-    
+
     # Load papers on app start
     demo.load(
         fn=lambda: paper_manager.get_current_page_papers(),
@@ -657,17 +628,10 @@ with demo:
     prev_button.click(paper_manager.prev_page, outputs=[paper_list])
     next_button.click(paper_manager.next_page, outputs=[paper_list])
 
-    # Sort option change: Toggle visibility of time_frame_dropdown based on sort method
-    sort_radio.change(
-        fn=lambda method: gr.update(visible=True) if method.lower() == "top" else gr.update(visible=False),
-        inputs=[sort_radio],
-        outputs=[time_frame_dropdown]
-    )
-
-    # Sort option change: Apply sorting method with time frame if applicable
+    # Sort option change: Apply sorting method
     sort_radio.change(
         fn=change_sort_method_ui,
-        inputs=[sort_radio, time_frame_dropdown],
+        inputs=[sort_radio, None],  # Pass None since time_frame_dropdown is removed
         outputs=[paper_list]
     )
 
@@ -683,4 +647,4 @@ with demo:
 # --- Launch the App ---
 
 if __name__ == "__main__":
-    demo.launch()
+    demo.launch()