Update modules/morphosyntax/morphosyntax_interface.py

modules/morphosyntax/morphosyntax_interface.py

@@ -23,6 +23,7 @@ from .morphosyntax_process import (
 )
 
 from ..utils.widget_utils import generate_unique_key
+
 from ..database.morphosyntax_iterative_mongo_db import (
     store_student_morphosyntax_base,
     store_student_morphosyntax_iteration,
@@ -37,12 +38,14 @@ logger = logging.getLogger(__name__)
 
 ###########################################################################
 def initialize_arc_analysis_state():
-    """Inicializa el estado del análisis de arcos y el caché si no existen"""
+    """Inicializa el estado del análisis de arcos y el caché si no existen."""
     if 'arc_analysis_state' not in st.session_state:
         st.session_state.arc_analysis_state = {
-            'base_id': None,
-            'original_text': '',
-            'iteration_text': '',
+            'base_id': None,           # ID del análisis base (ObjectId)
+            'original_text': '',       # Texto original
+            'original_analysis': None, # Dict con resultados del análisis base
+            'iteration_text': '',      # Texto de iteración
+            'iteration_analysis': None,# Dict con resultados de la iteración
             'analysis_count': 0
         }
         logger.info("Estado de análisis de arcos inicializado")
@@ -52,76 +55,88 @@ def initialize_arc_analysis_state():
         st.session_state.analysis_cache = {}
         logger.info("Caché de análisis inicializado")
 
-###########################################################################
+
 def reset_morpho_state():
-    """Resetea el estado del análisis morfosintáctico"""
+    """Resetea el estado del análisis morfosintáctico en sesión."""
     if 'arc_analysis_state' in st.session_state:
         st.session_state.arc_analysis_state = {
             'base_id': None,
             'original_text': '',
+            'original_analysis': None,
             'iteration_text': '',
+            'iteration_analysis': None,
             'analysis_count': 0
         }
 
-###########################################################################
+
 def display_original_analysis(container, analysis, lang_code, morpho_t):
-    """Muestra el análisis original en el contenedor especificado"""
+    """Muestra el análisis original en el contenedor especificado."""
     with container:
         st.subheader("Análisis Original")
         display_morphosyntax_results(analysis, lang_code, morpho_t)
 
+
 def display_iteration_analysis(container, analysis, lang_code, morpho_t):
-    """Muestra el análisis de cambios en el contenedor especificado"""
+    """Muestra el análisis de cambios en el contenedor especificado."""
     with container:
         st.subheader("Análisis de Cambios")
         display_morphosyntax_results(analysis, lang_code, morpho_t)
 
+
 def display_arc_diagram(doc, analysis):
-    """Muestra un diagrama de arco sin título"""
+    """Muestra un diagrama de arco sin título."""
     try:
         for sent in doc.sents:
-            rendered = displacy.render(sent, style="dep", options={
-                "distance": 100,
-                "arrow_spacing": 20,
-                "word_spacing": 30
-            })
-            
+            svg_html = displacy.render(
+                sent,
+                style="dep",
+                options={
+                    "distance": 100,
+                    "arrow_spacing": 20,
+                    "word_spacing": 30
+                }
+            )
             # Ajustar tamaño y posición
-            rendered = rendered.replace('height="375"', 'height="200"')
-            rendered = re.sub(
+            svg_html = svg_html.replace('height="375"', 'height="200"')
+            svg_html = re.sub(
                 r'<svg[^>]*>',
                 lambda m: m.group(0).replace('height="450"', 'height="300"'),
-                rendered
+                svg_html
             )
-            rendered = re.sub(
-                r'<g [^>]*transform="translate\((\d+),(\d+)\)"', 
+            svg_html = re.sub(
+                r'<g [^>]*transform="translate\((\d+),(\d+)\)"',
                 lambda m: f'<g transform="translate({m.group(1)},50)"',
-                rendered
+                svg_html
             )
             
             # Envolver en contenedor con estilo
-            rendered = f'<div class="arc-diagram-container">{rendered}</div>'
-            st.write(rendered, unsafe_allow_html=True)
+            svg_html = f'<div class="arc-diagram-container">{svg_html}</div>'
+            st.write(svg_html, unsafe_allow_html=True)
 
     except Exception as e:
         logger.error(f"Error en display_arc_diagram: {str(e)}")
 
-###########################################################################
+
 def cache_analysis_results(key, result):
-    """Almacena resultados de análisis en caché"""
+    """Almacena resultados de análisis en caché."""
     if 'analysis_cache' not in st.session_state:
-        st.session_state.analysis_cache = {}
+        initialize_arc_analysis_state()
     st.session_state.analysis_cache[key] = result
     logger.info(f"Resultado almacenado en caché con clave: {key}")
 
+
 def get_cached_analysis(key):
-    """Recupera resultados de análisis del caché"""
+    """Recupera resultados de análisis del caché."""
     if 'analysis_cache' not in st.session_state:
+        initialize_arc_analysis_state()
         return None
     return st.session_state.analysis_cache.get(key)
 
-###########################################################################
+
 def display_morphosyntax_interface(lang_code, nlp_models, morpho_t):
+    """
+    Interfaz principal para el análisis morfosintáctico.
+    """
     try:
         # CSS para layout estable
         st.markdown("""
@@ -149,46 +164,58 @@ def display_morphosyntax_interface(lang_code, nlp_models, morpho_t):
         # Inicializar estados si no existen
         initialize_arc_analysis_state()
 
-        # -- Tabs: definimos 3 pestañas
+        # Crear subtabs
         subtabs = st.tabs([
             "Análisis de Diagramas de Arco",
             "Análisis de Categorías",
             "Análisis Morfológico"
         ])
 
-        # =========== Pestaña 1: Diagrama de Arco ===============
+        # -------------------- Subtab 0: Diagramas de Arco --------------------
         with subtabs[0]:
-            col1, col2, col3 = st.columns([2,1,2])
+            # Botón de reset
+            col1, col2, col3 = st.columns([2, 1, 2])
             with col1:
                 if st.button("Nuevo Análisis", type="secondary", use_container_width=True):
                     reset_morpho_state()
-
-            # Contenedor de todo el proceso
-            with st.container():
-                # Input texto original
+                    # Forzar un refresco completo solo en este caso
+                    st.experimental_rerun()
+
+            # Container principal para análisis base
+            analysis_container = st.container()
+            with analysis_container:
+                # Entrada de texto original
+                text_input_key = f"original_text_{st.session_state.arc_analysis_state['analysis_count']}"
                 text_input = st.text_area(
-                    "Ingrese su texto",
+                    "Texto original",
                     value=st.session_state.arc_analysis_state.get('original_text', ''),
-                    key=f"original_text_{st.session_state.arc_analysis_state['analysis_count']}",
+                    key=text_input_key,
                     height=100
                 )
 
-                analyze_button = st.button(
-                    "Analizar Texto",
-                    type="primary",
-                    use_container_width=True
-                )
+                # Botón de análisis
+                col1, col2, col3 = st.columns([2, 1, 2])
+                with col1:
+                    analyze_button = st.button(
+                        "Analizar Texto",
+                        type="primary",
+                        use_container_width=True
+                    )
 
                 # Procesar texto original
                 if analyze_button and text_input.strip():
                     try:
+                        if not getattr(st.session_state, 'username', None):
+                            st.error("No se detectó username en session_state. Verifica tu login.")
+                            return
+
                         doc = nlp_models[lang_code](text_input)
                         analysis = perform_advanced_morphosyntactic_analysis(
                             text_input,
                             nlp_models[lang_code]
                         )
 
-                        # Guardar en Mongo
+                        # Guardar en BD (retornará ObjectId)
                         base_id = store_student_morphosyntax_base(
                             st.session_state.username,
                             text_input,
@@ -196,32 +223,47 @@ def display_morphosyntax_interface(lang_code, nlp_models, morpho_t):
                         )
 
                         if base_id:
-                            # Actualizar estado
-                            st.session_state.arc_analysis_state['base_id'] = base_id
-                            st.session_state.arc_analysis_state['original_text'] = text_input
-                            st.session_state.arc_analysis_state['analysis_count'] += 1
-
+                            # Actualizar el estado en session_state
+                            st.session_state.arc_analysis_state.update({
+                                'base_id': base_id,  # Guardamos el ObjectId
+                                'original_text': text_input,
+                                'original_analysis': analysis,
+                                'analysis_count': st.session_state.arc_analysis_state['analysis_count'] + 1,
+                                # Reiniciamos la iteración
+                                'iteration_text': '',
+                                'iteration_analysis': None
+                            })
+
+                            # Mostrar diagrama base
                             display_arc_diagram(doc, analysis)
 
+                            # Línea divisora
                             st.markdown('<hr class="divider">', unsafe_allow_html=True)
 
-                            # Caja de texto para la iteración
-                            st.subheader("Iteración / Cambios:")
-                            iteration_text = st.text_area(
-                                "Ingrese cambios o versión modificada",
-                                value=st.session_state.arc_analysis_state.get('iteration_text', ''),
-                                key="iteration_text",
-                                height=100
-                            )
-
-                            # Botón para analizar iteración
-                            iteration_button = st.button(
-                                "Analizar Cambios",
-                                type="primary",
-                                use_container_width=True
-                            )
-
-                            if iteration_button and iteration_text.strip():
+                            # Sección para la iteración
+                            st.subheader("Iteración / Cambios")
+                            with st.form("iteration_form"):
+                                # key para la iteración
+                                iteration_text_key = f"iteration_text_{st.session_state.arc_analysis_state['analysis_count']}"
+                                
+                                # En lugar de default = text_input, ahora dejamos vacío por defecto
+                                iteration_text = st.text_area(
+                                    "Texto de iteración",
+                                    value=st.session_state.arc_analysis_state.get('iteration_text', ''),
+                                    key=iteration_text_key,
+                                    height=100
+                                )
+
+                                # Botón de submit en el formulario
+                                col1, col2, col3 = st.columns([2,1,2])
+                                with col1:
+                                    submitted = st.form_submit_button(
+                                        "Analizar Cambios",
+                                        type="primary",
+                                        use_container_width=True
+                                    )
+
+                            if submitted and iteration_text.strip():
                                 try:
                                     doc_iter = nlp_models[lang_code](iteration_text)
                                     analysis_iter = perform_advanced_morphosyntactic_analysis(
@@ -231,15 +273,19 @@ def display_morphosyntax_interface(lang_code, nlp_models, morpho_t):
 
                                     iteration_id = store_student_morphosyntax_iteration(
                                         st.session_state.username,
-                                        base_id,
-                                        text_input,
-                                        iteration_text,
+                                        base_id,           # este es ObjectId
+                                        text_input,        # Texto original
+                                        iteration_text,    # Texto de iteración
                                         analysis_iter['arc_diagrams']
                                     )
 
                                     if iteration_id:
-                                        # Guardamos el texto de iteración en session_state
-                                        st.session_state.arc_analysis_state['iteration_text'] = iteration_text
+                                        # Actualizamos el estado de la iteración
+                                        st.session_state.arc_analysis_state.update({
+                                            'iteration_text': iteration_text,
+                                            'iteration_analysis': analysis_iter
+                                        })
+                                        # Mostrar diagrama de iteración
                                         display_arc_diagram(doc_iter, analysis_iter)
 
                                 except Exception as e:
@@ -248,59 +294,63 @@ def display_morphosyntax_interface(lang_code, nlp_models, morpho_t):
 
                     except Exception as e:
                         st.error("Error procesando análisis base")
-                        logger.error(f"Error: {str(e)}")
+                        logger.error(f"Error base: {str(e)}")
 
-        # =========== Pestaña 2: Categorías (placeholder) ============
+        # -------------------- Subtab 1: Análisis de Categorías ----------------
         with subtabs[1]:
             st.info("Análisis de Categorías en desarrollo...")
 
-        # =========== Pestaña 3: Análisis Morfológico (placeholder) ============
+        # -------------------- Subtab 2: Análisis Morfológico ------------------
         with subtabs[2]:
             st.info("Análisis Morfológico en desarrollo...")
 
     except Exception as e:
-        st.error("Error en la interfaz")
+        st.error("Error en la interfaz de morfosintaxis")
         logger.error(f"Error general en la interfaz: {str(e)}")
 
-###########################################################################
+
 def display_morphosyntax_results(result, lang_code, morpho_t):
     """
     Muestra solo el diagrama de arco.
     Args:
         result: Diccionario con el documento procesado y su análisis
         lang_code: Código del idioma
-        morpho_t: Diccionario de traducciones
+        morpho_t: Diccionario de traducciones (opcional)
     """
-    if result is None:
+    if not result:
         return
-        
     try:
         doc = result['doc']
         sentences = list(doc.sents)
         for i, sent in enumerate(sentences):
             try:
                 st.subheader(f"{morpho_t.get('sentence', 'Sentence')} {i+1}")
-                rendered = displacy.render(sent, style="dep", options={
-                    "distance": 100,
-                    "arrow_spacing": 20,
-                    "word_spacing": 30
-                })
-                rendered = rendered.replace('height="375"', 'height="200"')
-                rendered = re.sub(
+                svg_html = displacy.render(
+                    sent,
+                    style="dep",
+                    options={
+                        "distance": 100,
+                        "arrow_spacing": 20,
+                        "word_spacing": 30
+                    }
+                )
+                svg_html = svg_html.replace('height="375"', 'height="200"')
+                svg_html = re.sub(
                     r'<svg[^>]*>',
                     lambda m: m.group(0).replace('height="450"', 'height="300"'),
-                    rendered
+                    svg_html
                 )
-                rendered = re.sub(
+                svg_html = re.sub(
                     r'<g [^>]*transform="translate\((\d+),(\d+)\)"',
                     lambda m: f'<g transform="translate({m.group(1)},50)"',
-                    rendered
+                    svg_html
                 )
-                rendered = f'<div class="arc-diagram-container">{rendered}</div>'
-                st.write(rendered, unsafe_allow_html=True)
-            except Exception as e:
-                logger.error(f"Error en diagrama {i}: {str(e)}")
+                svg_html = f'<div class="arc-diagram-container">{svg_html}</div>'
+                st.write(svg_html, unsafe_allow_html=True)
+            except Exception as exc:
+                logger.error(f"Error mostrando diagrama de la oración {i}: {str(exc)}")
                 continue
     except Exception as e:
         logger.error(f"Error en display_morphosyntax_results: {str(e)}")
 
+