Update chatbot.py

chatbot.py

@@ -81,6 +81,13 @@ class Comsatsbot:
         return "success"
 
     def generate_response(self, question, history, context):
+        comsats_context = """
+        Comsats Attock Campus provides BS Computer Science, BS Software Engineering, BS Artificial Intelligence, BS English, BS Math, BS Electrical Engineering, BS Computer Engineering, BS BBA.
+        It has three departments: CS (CS, AI, SE), Math (Math, BBA, English), and EE (EE, CE).
+        It has a cricket ground, football ground, and two canteens. The first canteen is near the Math and EE departments, and the second canteen is near the CS department. There is also a mosque near the CS department. The CS department has theater-like rooms (LT), and there are a total of 9 theaters called LT. The Math department has classrooms (CR), and the EE department has labs.
+        They accept the NTS test for admission and provide the CGPA for 4 on 85 percent, and 3.66 between 79 to 84, and many more.
+        """
+
         prompt = f'''
         Kindly use the proper emojis where we need to use in responses.       
         You are a comsats assistant to help the user with comsats university-related queries. Your answer should be very concise and to the point with a short answer. Don't repeat irrelevant text. 
@@ -90,11 +97,11 @@ class Comsatsbot:
         Don't mention "according to provided context/Based on the provided chat history" in the response, just generate the response like a human without mentioning context and chat history.
         You are a conversational and helpful agent to help the comsats university attock campus students, and your task is to provide concise and direct answers to the questions.
         Your task is to use the emoji when there is a happy, sad, surprise, or angry expression required in the response. Kindly analyze the question and if an expression is required in response, then use the emoji. Otherwise, don't use the emoji, and remember that you don't need to use the emoji in simple studies and comsats-related questions.
-        For example, if the user asks the same question again and again, doesn't understand anything, asks wrong questions, etc., then use the emoji in such responses. Be choosy when using emojis in responses according to the user's question.
+        For example, if the user asks the same question again and again and doesn't understand anything, asks wrong questions, etc., then use the emoji in such responses. Be choosy when using emojis in responses according to the user's question.
         If there is any need to provide a URL, generate the URL according to the following structure. Kindly provide the link clickable. Your provided link should be generated according to this structure:
         [Click here to visit "website name"](website url "https://comsats.edu.pk") (Write the same URL as it is provided in the context below and don't use "www" in the URL.)
         Don't explain or repeat the prompt in the response.
-        1. Kindly generate a full concise answer and if you don't find the answer from context and chat history, then don’t make the answer, just say "I don’t know."
+        1. Kindly generate a full concise answer and if you don’t find the answer from context and chat history, then don’t make the answer, just say "I don’t know."
         2. Don’t explain irrelevant explanations and use the proper emojis in the answer if required.
         3. Always respond in a human-like tone and keep your answers concise, to the point, and friendly.
         4. If the question is conversational (like greetings, need any conversation, help related to studies, knowledge-based question, etc.), respond in a warm, conversational tone.
@@ -105,34 +112,37 @@ class Comsatsbot:
         Question: {question}
         Kindly answer the question from the provided context, and if you don’t find the answer from chat history and context, then inform the user "I don’t know" and don’t make the answer yourself.
         Use the following context to answer and don’t mention that you are answering from this context:
-        Comsats Attock Campus Provides BS Computer Science, BS Software Engineering, BS Artificial Intelligence, BS English, BS Math, BS Electrical Engineering, BS Computer Engineering, BS BBA.
-        It has three departments: CS (CS, AI, SE), Math (Math, BBA, English), and EE (EE, CE).
-        It has a cricket ground, football ground, and two canteens. The first canteen is near the Math and EE departments, and the second canteen is near the CS department. There is also a mosque near the CS department. The CS department has theater-like rooms (LT), and there are a total of 9 theaters called LT. The Math department has classrooms (CR), and the EE department has labs.
-        They accept the NTS test for admission and provide the CGPA for 4 on 85 percent, and 3.66 between 79 to 84, and many more.
-        {context}
+        {comsats_context}
         Context is ending.
         Now here is the chat history that you have to consider and identify
         **Consider the following chat history for additional context to answer the question:** 
         {history}
         Answer the following Question: {question}
         '''
+        
+        # Log context to check if it's passed correctly
+        print("Context being passed to the model:", comsats_context)
+        print("Question being asked:", question)
+
+        # Groq API call here
         while True:
             for api_key in self.api_keys:
                 self.client = Groq(api_key=api_key)
                 for model in self.models:
                     try:
                         chat_completion = self.client.chat.completions.create(
-                            messages=[
-                                {"role": "system", "content": prompt},
-                                {"role": "user", "content": f"Answer the following question: {question}"}
-                            ],
+                            messages=[{"role": "system", "content": prompt},
+                                      {"role": "user", "content": f"Answer the following question: {question}"}],
                             model=model,
                             max_tokens=1024,
                         )
+                        print("Response from Groq model:", chat_completion.choices[0].message.content)
                         return chat_completion.choices[0].message.content
-                    except Exception:
+                    except Exception as e:
+                        print(f"Error encountered: {e}")
                         time.sleep(2)
                         continue
+
         return "Sorry, unable to provide an answer at this time."
 
     def detect_language(self, question):
@@ -141,26 +151,25 @@ class Comsatsbot:
             for model in self.models:
                 try:
                     chat_completion = self.client.chat.completions.create(
-                        messages=[
-                            {
-                                "role": "system",
-                                "content": """
-                                    You are an expert agent, and your task is to detect the language. 
-                                    Return a JSON: {'detected_language': 'urdu' or 'english'}
-                                """
-                            },
+                        messages=[{
+                            "role": "system",
+                            "content": """
+                                You are an expert agent, and your task is to detect the language. 
+                                Return a JSON: {'detected_language': 'urdu' or 'english'}
+                            """
+                        },
                             {
-                                "role": "user",
-                                "content": f"Detect the language for: {question}"
-                            }
-                        ],
+                            "role": "user",
+                            "content": f"Detect the language for: {question}"
+                        }],
                         model=model,
                         max_tokens=256,
                         response_format={"type": "json_object"},
                     )
                     response = json.loads(chat_completion.choices[0].message.content)
                     return response['detected_language'].lower()
-                except Exception:
+                except Exception as e:
+                    print(f"Error encountered: {e}")
                     time.sleep(2)
                     continue
         return "english"
@@ -171,45 +180,23 @@ class Comsatsbot:
             for model in self.models:
                 try:
                     chat_completion = self.client.chat.completions.create(
-                        messages=[
+                        messages=[{
+                            "role": "system",
+                            "content": """
+                                Translate the following text into proper Urdu. Return a response without any extra explanation.
+                            """
+                        },
                             {
-                                "role": "system",
-                                "content": """
-                                    Translate the following text into proper Urdu. Return a JSON: 
-                                    {'text': 'translated urdu text'}
-                                """
-                            },
-                            {
-                                "role": "user",
-                                "content": f"Translate this: {text}"
-                            }
-                        ],
+                            "role": "user",
+                            "content": f"Translate this into Urdu: {text}"
+                        }],
                         model=model,
                         max_tokens=512,
-                        response_format={"type": "json_object"},
                     )
-                    response = json.loads(chat_completion.choices[0].message.content)
-                    return response['text']
-                except Exception:
+                    response = chat_completion.choices[0].message.content
+                    return response
+                except Exception as e:
+                    print(f"Error encountered: {e}")
                     time.sleep(2)
                     continue
-        return text
-
-    def response(self, question, chat_id):
-        chat_history = self.load_chat(chat_id)
-
-        for entry in chat_history:
-            self.memory.save_context({"input": entry["question"]}, {"output": entry["answer"]})
-
-        language = self.detect_language(question)
-
-        if language == 'urdu':
-            question_translation = GoogleTranslator(source='ur', target='en').translate(question)
-            context = self.faiss_retriever.invoke(question_translation)
-        else:
-            context = self.faiss_retriever.invoke(question)
-
-        answer = self.generate_response(question, chat_history, context)
-
-        self.update_chat(chat_id, question, answer)
-        return answer
+        return text