Update app.py

app.py

@@ -53,6 +53,16 @@ def handle_errors(func):
             st.rerun()
     return wrapper
 
+def show_progress(message):
+    progress_bar = st.progress(0)
+    status_text = st.empty()
+    for i in range(100):
+        time.sleep(0.02)
+        progress_bar.progress(i + 1)
+        status_text.text(f"{message}... {i+1}%")
+    progress_bar.empty()
+    status_text.empty()
+
 def scroll_to_bottom():
     ctx = get_script_run_ctx()
     if ctx and runtime.exists():
@@ -73,168 +83,31 @@ def summarize_pdf(_pdf_file_path, num_clusters=10):
     embeddings_model = OpenAIEmbeddings(model="text-embedding-3-small", api_key=openai_api_key)
     llm = ChatOpenAI(model="gpt-4", api_key=openai_api_key, temperature=0.3)
     
-    # Load PDF with page numbers
-    loader = PyMuPDFLoader(_pdf_file_path)
-    docs = loader.load()
-    
-    # Create chunks with page metadata
-    text_splitter = SpacyTextSplitter(chunk_size=500)
-    chunks_with_metadata = []
-    for doc in docs:
-        chunks = text_splitter.split_text(doc.page_content)
-        for chunk in chunks:
-            chunks_with_metadata.append({
-                "text": clean_text(chunk),
-                "page": doc.metadata["page"] + 1  # Convert to 1-based numbering
-            })
-    
-    # Prepare prompt with citation instructions
     prompt = ChatPromptTemplate.from_template(
-        """Generate a comprehensive summary with inline citations using [Source X] format. 
-        Include these elements:
+        """Generate a comprehensive summary with these elements:
         1. Key findings and conclusions
         2. Main methodologies used
         3. Important data points
         4. Limitations mentioned
-        
-        Structure your response as:
-        ## Comprehensive Summary
-        {summary_content}
-        
-        Contexts: {topic}"""
+        Context: {topic}"""
     )
     
-    # Generate summary
-    chain = prompt | llm | StrOutputParser()
-    raw_summary = chain.invoke({
-        "topic": ' '.join([chunk["text"] for chunk in chunks_with_metadata])
-    })
-    
-    return generate_interactive_citations(raw_summary, chunks_with_metadata)
-
-def generate_interactive_citations(summary_text, source_chunks):
-    # Create source entries with page numbers and full text
-    sources_html = """<div style="margin-top: 2rem; padding-top: 1rem; border-top: 1px solid #e0e0e0;">
-                        <h3 style="color: #2c3e50;">📖 Source References</h3>"""
-    
-    source_mapping = {}
-    for idx, chunk in enumerate(source_chunks):
-        source_id = f"source-{idx+1}"
-        source_mapping[idx+1] = {
-            "id": source_id,
-            "page": chunk["page"],
-            "text": chunk["text"]
-        }
-        
-        sources_html += f"""
-        <div id="{source_id}" style="margin: 1rem 0; padding: 1rem; 
-                    border: 1px solid #e0e0e0; border-radius: 8px;
-                    background-color: #f8f9fa; transition: all 0.3s ease;">
-            <div style="display: flex; justify-content: space-between; align-items: center;">
-                <div style="font-weight: 600; color: #4CAF50;">Source {idx+1}</div>
-                <div style="font-size: 0.9em; color: #666;">Page {chunk['page']}</div>
-            </div>
-            <div style="margin-top: 0.5rem; color: #444; font-size: 0.95em;">
-                {chunk["text"]}
-            </div>
-        </div>
-        """
-    
-    sources_html += "</div>"
-    
-    # Add click interactions
-    interaction_js = """
-    <script>
-    document.querySelectorAll('.citation-link').forEach(item => {
-        item.addEventListener('click', function(e) {
-            e.preventDefault();
-            const sourceId = this.getAttribute('data-source');
-            const sourceDiv = document.getElementById(sourceId);
-            
-            // Highlight animation
-            sourceDiv.style.transform = 'scale(1.02)';
-            sourceDiv.style.boxShadow = '0 4px 12px rgba(76,175,80,0.2)';
-            
-            setTimeout(() => {
-                sourceDiv.style.transform = 'none';
-                sourceDiv.style.boxShadow = 'none';
-            }, 500);
-            
-            // Smooth scroll
-            sourceDiv.scrollIntoView({behavior: 'smooth', block: 'start'});
-        });
-    });
-    </script>
-    """
-    
-    # Replace citations with interactive links
-    cited_summary = re.sub(r'\[Source (\d+)\]', 
-        lambda m: f'<a class="citation-link" data-source="source-{m.group(1)}" '
-                  f'style="cursor: pointer; color: #4CAF50; text-decoration: none; '
-                  f'border-bottom: 1px dashed #4CAF50;">[Source {m.group(1)}]</a>', 
-        summary_text)
-    
-    return f"""
-    <div style="margin-bottom: 3rem;">
-        {cited_summary}
-        {sources_html}
-    </div>
-    {interaction_js}
-    """
-
-@st.cache_data(show_spinner=False, ttl=3600)
-@handle_errors
-def qa_pdf(_pdf_file_path, query, num_clusters=5):
-    embeddings_model = OpenAIEmbeddings(model="text-embedding-3-small", api_key=openai_api_key)
-    llm = ChatOpenAI(model="gpt-4", api_key=openai_api_key, temperature=0.3)
-    
-    # Load PDF with page numbers
     loader = PyMuPDFLoader(_pdf_file_path)
     docs = loader.load()
+    full_text = "\n".join(doc.page_content for doc in docs)
+    cleaned_full_text = clean_text(remove_references(full_text))
     
-    # Create chunks with page metadata
     text_splitter = SpacyTextSplitter(chunk_size=500)
-    chunks_with_metadata = []
-    for doc in docs:
-        chunks = text_splitter.split_text(doc.page_content)
-        for chunk in chunks:
-            chunks_with_metadata.append({
-                "text": clean_text(chunk),
-                "page": doc.metadata["page"] + 1
-            })
-    
-    # Find relevant chunks
-    embeddings = embeddings_model.embed_documents([chunk["text"] for chunk in chunks_with_metadata])
-    query_embedding = embeddings_model.embed_query(query)
-    similarities = cosine_similarity([query_embedding], embeddings)[0]
-    top_indices = np.argsort(similarities)[-num_clusters:]
+    split_contents = text_splitter.split_text(cleaned_full_text)
     
-    # Prepare prompt with citation instructions
-    prompt = ChatPromptTemplate.from_template(
-        """Answer this question with inline citations using [Source X] format:
-        {question}
-        
-        Use these verified sources:
-        {context}
-        
-        Structure your answer with:
-        - Clear section headings
-        - Bullet points for lists
-        - Citations for all factual claims"""
-    )
+    embeddings = embeddings_model.embed_documents(split_contents)
+    kmeans = KMeans(n_clusters=num_clusters, random_state=0).fit(embeddings)
+    closest_indices = [np.argmin(np.linalg.norm(embeddings - center, axis=1)) 
+                      for center in kmeans.cluster_centers_]
     
     chain = prompt | llm | StrOutputParser()
-    raw_answer = chain.invoke({
-        "question": query,
-        "context": '\n\n'.join([f"Source {i+1} (Page {chunks_with_metadata[i]['page']}): {chunks_with_metadata[i]['text']}" 
-                              for i in top_indices])
-    })
-    
-    return generate_interactive_citations(raw_answer, [chunks_with_metadata[i] for i in top_indices])
+    return chain.invoke({"topic": ' '.join([split_contents[idx] for idx in closest_indices])})
 
-# (Keep the rest of the code from previous implementation for PDF processing and UI)
-# [Include the process_pdf, image_to_base64, and Streamlit UI code from previous response]
-# [Make sure to maintain all the UI improvements and error handling]
 @st.cache_data(show_spinner=False, ttl=3600)
 @handle_errors
 def qa_pdf(_pdf_file_path, query, num_clusters=5):
@@ -310,73 +183,36 @@ def image_to_base64(img):
     img.save(buffered, format="JPEG", quality=85)
     return base64.b64encode(buffered.getvalue()).decode()
 
-# Streamlit UI Configuration
+# Streamlit UI
 st.set_page_config(
-    page_title="PDF Research Assistant",
+    page_title="PDF Assistant",
     page_icon="📄",
     layout="wide",
     initial_sidebar_state="expanded"
 )
 
-# Custom CSS Styles
-st.markdown("""
-<style>
-    .citation-link {
-        transition: all 0.2s ease;
-        font-weight: 500;
-    }
-    .citation-link:hover {
-        color: #45a049 !important;
-        border-bottom-color: #45a049 !important;
-    }
-    .stChatMessage {
-        border-radius: 12px;
-        box-shadow: 0 4px 12px rgba(0,0,0,0.08);
-        margin: 1.5rem 0;
-        padding: 1.5rem;
-    }
-    .stButton>button {
-        background: linear-gradient(135deg, #4CAF50, #45a049);
-        transition: transform 0.2s ease, box-shadow 0.2s ease;
-    }
-    .stButton>button:hover {
-        transform: translateY(-1px);
-        box-shadow: 0 4px 12px rgba(76,175,80,0.3);
-    }
-    [data-testid="stFileUploader"] {
-        border: 2px dashed #4CAF50;
-        border-radius: 12px;
-        background: #f8fff8;
-    }
-</style>
-""", unsafe_allow_html=True)
-
-# Session state initialization
 if 'chat_history' not in st.session_state:
     st.session_state.chat_history = []
 if 'current_file' not in st.session_state:
     st.session_state.current_file = None
 
-# Main UI
-st.title("📄 Academic PDF Analyzer")
+st.title("📄 Smart PDF Analyzer")
 st.markdown("""
-<div style="border-left: 4px solid #4CAF50; padding-left: 1.5rem; margin: 2rem 0;">
-    <p style="color: #2c3e50; font-size: 1.1rem;">🔍 Upload research papers to:
-    <ul style="color: #2c3e50; font-size: 1rem;">
-        <li>Generate citations-backed summaries</li>
-        <li>Trace claims to original sources</li>
-        <li>Extract data tables and figures</li>
-        <li>Q&A with verifiable references</li>
+<div style="border-left: 4px solid #4CAF50; padding-left: 1rem; margin: 1rem 0;">
+    <p style="color: #666; font-size: 0.95rem;">✨ Upload a PDF to:
+    <ul style="color: #666; font-size: 0.95rem;">
+        <li>Generate structured summaries</li>
+        <li>Extract visual content</li>
+        <li>Ask contextual questions</li>
     </ul>
     </p>
 </div>
 """, unsafe_allow_html=True)
 
-# File uploader
 uploaded_file = st.file_uploader(
-    "Upload research PDF", 
+    "Choose PDF file", 
     type="pdf",
-    help="Maximum file size: 50MB",
+    help="Max file size: 50MB",
     on_change=lambda: setattr(st.session_state, 'chat_history', [])
 )
 
@@ -384,13 +220,11 @@ if uploaded_file and uploaded_file.size > MAX_FILE_SIZE:
     st.error("File size exceeds 50MB limit")
     st.stop()
 
-# Document processing
 if uploaded_file:
     file_path = tempfile.NamedTemporaryFile(delete=False).name
     with open(file_path, "wb") as f:
         f.write(uploaded_file.getbuffer())
     
-    # Chat interface
     chat_container = st.container()
     with chat_container:
         for idx, chat in enumerate(st.session_state.chat_history):
@@ -403,28 +237,80 @@ if uploaded_file:
                     message(chat["bot"], key=f"bot_{idx}", allow_html=True)
         scroll_to_bottom()
 
-    # Interaction controls
     with st.container():
         col1, col2, col3 = st.columns([3, 2, 2])
         with col1:
-            user_input = st.chat_input("Ask a research question...")
+            user_input = st.chat_input("Ask about the document...")
         with col2:
-            if st.button("📄 Generate Summary", use_container_width=True):
+            if st.button("📝 Generate Summary", use_container_width=True):
                 with st.spinner("Analyzing document structure..."):
+                    show_progress("Generating summary")
                     summary = summarize_pdf(file_path)
                     st.session_state.chat_history.append({
-                        "bot": f"## Research Summary\n{summary}"
+                        "user": "Summary request", 
+                        "bot": f"## Document Summary\n{summary}"
                     })
                     st.rerun()
         with col3:
-            if st.button("🔄 Clear Session", use_container_width=True):
-                st.session_state.chat_history = []
-                st.rerun()
+            if st.button("🖼️ Extract Visuals", use_container_width=True):
+                with st.spinner("Identifying figures and tables..."):
+                    show_progress("Extracting visuals")
+                    figures, tables = process_pdf(file_path)
+                    if figures:
+                        st.session_state.chat_history.append({
+                            "bot": f"Found {len(figures)} figures:"
+                        })
+                        for fig in figures:
+                            st.session_state.chat_history.append({
+                                "bot": f'<img src="data:image/jpeg;base64,{image_to_base64(fig)}" style="max-width: 100%;">'
+                            })
+                    if tables:
+                        st.session_state.chat_history.append({
+                            "bot": f"Found {len(tables)} tables:"
+                        })
+                        for tab in tables:
+                            st.session_state.chat_history.append({
+                                "bot": f'<img src="data:image/jpeg;base64,{image_to_base64(tab)}" style="max-width: 100%;">'
+                            })
+                    st.rerun()
 
-    # Handle user questions
     if user_input:
         st.session_state.chat_history.append({"user": user_input})
-        with st.spinner("Verifying sources..."):
+        with st.spinner("Analyzing query..."):
+            show_progress("Generating answer")
             answer = qa_pdf(file_path, user_input)
-            st.session_state.chat_history[-1]["bot"] = f"## Research Answer\n{answer}"
-            st.rerun()
+            st.session_state.chat_history[-1]["bot"] = f"## Answer\n{answer}"
+            st.rerun()
+
+st.markdown("""
+<style>
+    .stChatMessage {
+        padding: 1.25rem;
+        margin: 1rem 0;
+        border-radius: 12px;
+        box-shadow: 0 2px 8px rgba(0,0,0,0.1);
+        transition: transform 0.2s ease;
+    }
+    .stChatMessage:hover {
+        transform: translateY(-2px);
+    }
+    .stButton>button {
+        background: linear-gradient(45deg, #4CAF50, #45a049);
+        color: white;
+        border: none;
+        border-radius: 8px;
+        padding: 12px 24px;
+        font-size: 16px;
+        transition: all 0.3s ease;
+    }
+    .stButton>button:hover {
+        box-shadow: 0 4px 12px rgba(76,175,80,0.3);
+        transform: translateY(-1px);
+    }
+    [data-testid="stFileUploader"] {
+        border: 2px dashed #4CAF50;
+        border-radius: 12px;
+        padding: 2rem;
+    }
+</style>
+""", unsafe_allow_html=True)