Update app.py

app.py

@@ -65,23 +65,27 @@ def load_pdf(file_path):
 
 def load_chatbot():
     return load_qa_chain(llm=OpenAI(), chain_type="stuff") 
+
+def display_chat_history(chat_history):
+    for chat in chat_history:
+        background_color = "#FFA07A" if chat[2] == "new" else "#acf" if chat[0] == "User" else "#caf"
+        st.markdown(f"<div style='background-color: {background_color}; padding: 10px; border-radius: 10px; margin: 10px;'>{chat[0]}: {chat[1]}</div>", unsafe_allow_html=True)
+
 def main():
     st.title("BinDocs Chat App")
 
-
     if "chat_history" not in st.session_state:
         st.session_state['chat_history'] = []
 
-    if "current_input" not in st.session_state:
-        st.session_state['current_input'] = ""
-
     display_chat_history(st.session_state['chat_history'])
 
     st.write("<!-- Start Spacer -->", unsafe_allow_html=True)
     st.write("<div style='flex: 1;'></div>", unsafe_allow_html=True)
     st.write("<!-- End Spacer -->", unsafe_allow_html=True)
-
     
+    new_messages_placeholder = st.empty()
+    loading_message = st.empty()
+
     pdf = st.file_uploader("Upload your PDF", type="pdf")
 
     if pdf is not None:
@@ -91,7 +95,6 @@ def main():
             st.session_state['last_input'] = query  # Save the current query as the last input
             st.session_state['chat_history'].append(("User", query, "new"))
 
-            loading_message = st.empty()
             loading_message.text('Bot is thinking...')
 
             VectorStore = load_pdf(pdf)
@@ -102,28 +105,22 @@ def main():
 
             st.session_state['chat_history'].append(("Bot", response, "new"))
 
-        # Display new messages at the bottom
-        new_messages = st.session_state['chat_history'][-2:]
-        for chat in new_messages:
-            background_color = "#FFA07A" if chat[2] == "new" else "#acf" if chat[0] == "User" else "#caf"
-            new_messages_placeholder.markdown(f"<div style='background-color: {background_color}; padding: 10px; border-radius: 10px; margin: 10px;'>{chat[0]}: {chat[1]}</div>", unsafe_allow_html=True)
-
-        # Scroll to the latest response using JavaScript
-        st.write("<script>document.getElementById('response').scrollIntoView();</script>", unsafe_allow_html=True)
-
-        loading_message.empty()
+            # Display new messages at the bottom
+            new_messages = st.session_state['chat_history'][-2:]
+            for chat in new_messages:
+                background_color = "#FFA07A" if chat[2] == "new" else "#acf" if chat[0] == "User" else "#caf"
+                new_messages_placeholder.markdown(f"<div style='background-color: {background_color}; padding: 10px; border-radius: 10px; margin: 10px;'>{chat[0]}: {chat[1]}</div>", unsafe_allow_html=True)
 
-        # Clear the input field by setting the query variable to an empty string
-        query = ""
+            # Scroll to the latest response using JavaScript
+            st.write("<script>document.getElementById('response').scrollIntoView();</script>", unsafe_allow_html=True)
 
-    # Mark all messages as old after displaying
-    st.session_state['chat_history'] = [(sender, msg, "old") for sender, msg, _ in st.session_state['chat_history']]
+            loading_message.empty()
 
+            # Clear the input field by setting the query variable to an empty string
+            query = ""
 
-def display_chat_history(chat_history):
-    for chat in chat_history:
-        background_color = "#FFA07A" if chat[2] == "new" else "#acf" if chat[0] == "User" else "#caf"
-        st.markdown(f"<div style='background-color: {background_color}; padding: 10px; border-radius: 10px; margin: 10px;'>{chat[0]}: {chat[1]}</div>", unsafe_allow_html=True)
+        # Mark all messages as old after displaying
+        st.session_state['chat_history'] = [(sender, msg, "old") for sender, msg, _ in st.session_state['chat_history']]
 
 if __name__ == "__main__":
-    main()
+    main()