Update app.py

app.py

@@ -528,71 +528,94 @@ if st.button("Generate Patent Insights"):
             # Start the timer
             start_time = time.time()
 
-            # Kick off the crew with user inputs
+            # Validate user inputs
             if not patent_area or not stakeholder:
                 st.error("Please provide both Patent Technology Area and Stakeholder.")
-            else:
-                logging.info(f"Starting analysis with Topic: {patent_area}, Stakeholder: {stakeholder}")
-                results = crew.kickoff(inputs={"topic": patent_area, "stakeholder": stakeholder})
+                st.stop()
+
+            # Log and execute the analysis
+            logging.info(f"Starting analysis with Topic: {patent_area}, Stakeholder: {stakeholder}")
+            results = crew.kickoff(inputs={"topic": patent_area, "stakeholder": stakeholder})
 
             # Calculate elapsed time
             elapsed_time = time.time() - start_time
 
-            # Extract Writer's Output
+            # Extract Outputs
+            planner_output = getattr(results.tasks_output[0], "raw", "No details available.")
+            analyst_output = getattr(results.tasks_output[1], "raw", "No details available.")
             writer_output = getattr(results.tasks_output[2], "raw", "No details available.")
-            if writer_output and writer_output.strip():
-                st.markdown("### Final Report")
-                st.write(writer_output)
-            else:
-                st.warning("No final report available.")
 
-            # Expandable section for detailed insights
-            with st.expander("Explore Detailed Insights"):
-                tab1, tab2 = st.tabs(["Planner's Insights", "Analyst's Analysis"])
+            # Initialize empty variables to avoid undefined errors
+            key_insights, data_insights, charts, table_data = [], [], [], []
 
-                # Planner's Insights
-                with tab1:
-                    planner_output = getattr(results.tasks_output[0], "raw", "No details available.")
-                    if planner_output and planner_output.strip():
-                        st.write(planner_output)
-                    else:
-                        st.warning("No planner insights available.")
+            # Parse Analyst Output (Key Insights + Data Insights)
+            if analyst_output and analyst_output.strip():
+                key_insights, data_insights = parse_analyst_output(analyst_output)
 
-                # Analyst's Analysis
-                with tab2:
-                    analyst_output = getattr(results.tasks_output[1], "raw", "No details available.")
-                    if analyst_output and analyst_output.strip():
-                        st.write(analyst_output)
+                # Generate visualizations and tables if enabled
+                if enable_advanced_analysis and data_insights:
+                    charts = create_visualizations(data_insights)
+                    table_data = display_table(data_insights)
+                else:
+                    st.info("No data insights available for visualizations.")
+
+            # Unified Tabs for Final Report, Planner, and Analyst Outputs
+            tab1, tab2, tab3 = st.tabs(["📄 Final Report", "📝 Planner's Insights", "📊 Analyst's Analysis"])
+
+            # Final Report Tab
+            with tab1:
+                st.subheader("📄 Final Report")
+                if writer_output and writer_output.strip():
+                    st.write(writer_output)
+                else:
+                    st.warning("No final report available.")
 
-                        # Parse Analyst Output (Key Insights + Data Insights)
-                        key_insights, data_insights = parse_analyst_output(analyst_output)
+            # Planner's Insights Tab
+            with tab2:
+                st.subheader("📝 Planner's Insights")
+                if planner_output and planner_output.strip():
+                    st.write(planner_output)
+                else:
+                    st.warning("No planner insights available.")
+
+            # Analyst's Analysis Tab
+            with tab3:
+                st.subheader("📊 Analyst's Analysis")
+                if analyst_output and analyst_output.strip():
+                    st.write(analyst_output)
+
+                    if data_insights:
                         st.subheader("Structured Analyst Output")
                         st.write(data_insights)
+                    else:
+                        st.info("No structured data insights available.")
 
-                        # Create Visualizations if enabled
-                        charts = []
-                        if enable_advanced_analysis and data_insights:
-                            charts = create_visualizations(data_insights)
-                        else:
-                            st.info("No data insights available for visualizations.")
-
-                        # Display Data Tables
-                        table_data = display_table(data_insights)
+                    if charts:
+                        st.subheader("Generated Visualizations")
+                        for chart_path in charts:
+                            st.image(chart_path, caption="Generated Chart", use_column_width=True)
+                    else:
+                        st.info("No charts available for display.")
 
+                    if table_data:
+                        st.subheader("Data Tables")
+                        st.write(table_data)
                     else:
-                        st.warning("No analyst analysis available.")
+                        st.info("No tables available for display.")
+                else:
+                    st.warning("No analyst analysis available.")
 
             # Notify user that the analysis is complete
-            st.success(f"Analysis completed in {elapsed_time:.2f} seconds.")
+            st.success(f"✅ Analysis completed in {elapsed_time:.2f} seconds.")
 
-            # Generate the PDF report with Key Insights and Data Insights
+            # PDF Report Generation
             if writer_output:
                 pdf_path = generate_pdf_report(
                     result=writer_output,
                     charts=charts,
                     table_data=data_insights,
                     metadata={"Technology Area": patent_area, "Stakeholder": stakeholder},
-                    key_insights=key_insights  # Pass key insights to the PDF
+                    key_insights=key_insights
                 )
 
                 # Download button for the generated PDF
@@ -604,9 +627,9 @@ if st.button("Generate Patent Insights"):
                         mime="application/pdf"
                     )
             else:
-                st.warning("Report generation skipped due to missing content.")
+                st.warning("⚠️ Report generation skipped due to missing content.")
 
         except Exception as e:
             error_message = traceback.format_exc()
             logging.error(f"An error occurred during execution:\n{error_message}")
-            st.error(f"⚠️ An unexpected error occurred:\n{e}")
+            st.error(f"⚠️ An unexpected error occurred:\n{e}")