HOTFIX: Improve subject line for data exports, clean up logs

ai_ta_backend/main.py

@@ -1,5 +1,4 @@
 import os
-import threading
 import time
 from typing import List
 
@@ -112,10 +111,7 @@ def getTopContexts(service: RetrievalService) -> Response:
         f"Missing one or more required parameters: 'search_query' and 'course_name' must be provided. Search query: `{search_query}`, Course name: `{course_name}`"
     )
 
-  print("NUM ACTIVE THREADS (top of getTopContexts):", threading.active_count())
-
   found_documents = service.getTopContexts(search_query, course_name, token_limit)
-  print("NUM ACTIVE THREADS (after instantiating Ingest() class in getTopContexts):", threading.active_count())
 
   response = jsonify(found_documents)
   response.headers.add('Access-Control-Allow-Origin', '*')

ai_ta_backend/service/export_service.py

@@ -43,8 +43,7 @@ def export_documents_json(self, course_name: str, from_date='', to_date=''):
       # background task of downloading data - map it with above ID
       executor = ProcessPoolExecutor()
       executor.submit(export_data_in_bg, response, "documents", course_name, s3_filepath)
-      return {"response": 'Download from S3', 
-              "s3_path": s3_filepath}
+      return {"response": 'Download from S3', "s3_path": s3_filepath}
 
     else:
       # Fetch data
@@ -96,8 +95,6 @@ def export_documents_json(self, course_name: str, from_date='', to_date=''):
       else:
         return {"response": "No data found between the given dates."}
 
-
-
   def export_convo_history_json(self, course_name: str, from_date='', to_date=''):
     """
 		This function exports the conversation history to a csv file.
@@ -169,6 +166,7 @@ def export_convo_history_json(self, course_name: str, from_date='', to_date=''):
 
 # Encountered pickling error while running the background task. So, moved the function outside the class.
 
+
 def export_data_in_bg(response, download_type, course_name, s3_path):
   """
 	This function is called in export_documents_csv() to upload the documents to S3.
@@ -184,7 +182,7 @@ def export_data_in_bg(response, download_type, course_name, s3_path):
 	"""
   s3 = AWSStorage()
   sql = SQLDatabase()
-  
+
   total_doc_count = response.count
   first_id = response.data[0]['id']
   print("total_doc_count: ", total_doc_count)
@@ -203,7 +201,7 @@ def export_data_in_bg(response, download_type, course_name, s3_path):
 
     # writing to file
     if not os.path.isfile(file_path):
-       df.to_json(file_path, orient='records', lines=True)
+      df.to_json(file_path, orient='records', lines=True)
     else:
       df.to_json(file_path, orient='records', lines=True, mode='a')
 
@@ -237,10 +235,7 @@ def export_data_in_bg(response, download_type, course_name, s3_path):
     #print("s3_url: ", s3_url)
 
     # get admin email IDs
-    headers = {
-      "Authorization": f"Bearer {os.environ['VERCEL_READ_ONLY_API_KEY']}",
-      "Content-Type": "application/json"
-    }
+    headers = {"Authorization": f"Bearer {os.environ['VERCEL_READ_ONLY_API_KEY']}", "Content-Type": "application/json"}
 
     hget_url = str(os.environ['VERCEL_BASE_URL']) + "course_metadatas/" + course_name
     response = requests.get(hget_url, headers=headers)
@@ -265,7 +260,12 @@ def export_data_in_bg(response, download_type, course_name, s3_path):
       return "No admin emails found. Email not sent."
 
     # send email to admins
-    subject = "UIUC.chat Data Export Complete for " + course_name
+    if download_type == "documents":
+      subject = "UIUC.chat Documents Export Complete for " + course_name
+    elif download_type == "conversations":
+      subject = "UIUC.chat Conversation History Export Complete for " + course_name
+    else:
+      subject = "UIUC.chat Export Complete for " + course_name
     body_text = "The data export for " + course_name + " is complete.\n\nYou can download the file from the following link: \n\n" + s3_url + "\n\nThis link will expire in 48 hours."
     email_status = send_email(subject, body_text, os.environ['EMAIL_SENDER'], admin_emails, bcc_emails)
     print("email_status: ", email_status)
@@ -274,4 +274,4 @@ def export_data_in_bg(response, download_type, course_name, s3_path):
 
   except Exception as e:
     print(e)
-    return "Error: " + str(e)
+    return "Error: " + str(e)

ai_ta_backend/utils/filtering_contexts.py

@@ -137,8 +137,6 @@
 #   langsmith_prompt_obj = filter_unrelated_contexts_zephyr
 #   posthog = Posthog(sync_mode=True, project_api_key=os.environ['POSTHOG_API_KEY'], host='https://app.posthog.com')
 
-#   print("NUM ACTIVE THREADS (top of filtering_contexts):", threading.active_count())
-
 #   max_concurrency = min(100, len(contexts))
 #   print("max_concurrency is max of 100, or len(contexts), whichever is less ---- Max concurrency:", max_concurrency)
 #   print("Num contexts to filter:", len(contexts))
@@ -153,14 +151,11 @@
 #                                      timeout=timeout,
 #                                      fetch_local=False)
 
-#   print("NUM ACTIVE THREADS (before cleanup filtering_contexts):", threading.active_count())
 #   # Cleanup
 #   for task in in_progress:
 #     ray.cancel(task)
 #   results = ray.get(done_tasks)
-#   print("NUM ACTIVE THREADS (before kill filtering_contexts):", threading.active_count())
 #   ray.kill(actor)
-#   print("NUM ACTIVE THREADS (after kill filtering_contexts):", threading.active_count())
 
 #   best_contexts_to_keep = [
 #       r['context'] for r in results if r and 'context' in r and 'completion' in r and parse_result(r['completion'])