Update modules/studentact/current_situation_interface.py

modules/studentact/current_situation_interface.py

@@ -97,14 +97,15 @@ def display_current_situation_interface(lang_code, nlp_models, t):
     """
     TAB:
       - Expander con radio para tipo de texto
-    Contenedor-1 (2 filas con borde):
-       - Fila 1: Métricas base
-       - Fila 2: Métricas iteración
+    Contenedor-1 con expanders:
+       - Expander "Métricas de la línea base"
+       - Expander "Métricas de la iteración"
     Contenedor-2 (2 columnas):
        - Col1: Texto base
        - Col2: Texto iteración
-    Al final, Recomendaciones en un expander (una sola fila).
+    Al final, Recomendaciones en un expander (una sola “fila”).
     """
+
     # --- Inicializar session_state ---
     if 'base_text' not in st.session_state:
         st.session_state.base_text = ""
@@ -135,42 +136,26 @@ def display_current_situation_interface(lang_code, nlp_models, t):
         st.markdown("---")
 
         # ---------------------------------------------------------------------
-        # CONTENEDOR-1: Métricas (2 filas con borde)
+        # CONTENEDOR-1: Expanders para métricas base e iteración
         # ---------------------------------------------------------------------
         with st.container():
-            # Fila 1: Métricas de la línea base
-            st.markdown(
-                """
-                <div style="border:1px solid black; padding:10px; margin-bottom:10px;">
-                    <p style="font-weight:bold;">Métricas de la línea base</p>
-                """,
-                unsafe_allow_html=True
-            )
-
-            # Si se presionó "Analizar Base", se muestran los valores
-            # De lo contrario, mostramos la maqueta vacía
-            if st.session_state.show_base and st.session_state.base_metrics:
-                display_metrics_in_one_row(st.session_state.base_metrics, text_type)
-            else:
-                display_empty_metrics_row()
-
-            st.markdown("</div>", unsafe_allow_html=True)
-
-            # Fila 2: Métricas de iteración
-            st.markdown(
-                """
-                <div style="border:1px solid black; padding:10px; margin-bottom:10px;">
-                    <p style="font-weight:bold;">Métricas de la iteración</p>
-                """,
-                unsafe_allow_html=True
-            )
+            # --- Expander para la línea base ---
+            with st.expander("Métricas de la línea base", expanded=False):
+                if st.session_state.show_base and st.session_state.base_metrics:
+                    # Mostramos los valores reales
+                    display_metrics_in_one_row(st.session_state.base_metrics, text_type)
+                else:
+                    # Mostramos la maqueta vacía
+                    display_empty_metrics_row()
 
-            if st.session_state.show_iter and st.session_state.iter_metrics:
-                display_metrics_in_one_row(st.session_state.iter_metrics, text_type)
-            else:
-                display_empty_metrics_row()
+            # --- Expander para la iteración ---
+            with st.expander("Métricas de la iteración", expanded=False):
+                if st.session_state.show_iter and st.session_state.iter_metrics:
+                    display_metrics_in_one_row(st.session_state.iter_metrics, text_type)
+                else:
+                    display_empty_metrics_row()
 
-            st.markdown("</div>", unsafe_allow_html=True)
+        st.markdown("---")
 
         # ---------------------------------------------------------------------
         # CONTENEDOR-2: 2 columnas (texto base | texto iteración)
@@ -178,13 +163,14 @@ def display_current_situation_interface(lang_code, nlp_models, t):
         with st.container():
             col_left, col_right = st.columns(2)
 
+            # Columna izquierda: Texto base
             with col_left:
                 st.markdown("**Texto base**")
                 text_base = st.text_area(
                     label="",
                     value=st.session_state.base_text,
                     key="text_base_area",
-                    placeholder="Pega aquí tu texto base"
+                    placeholder="Pega aquí tu texto base",
                 )
                 if st.button("Analizar Base"):
                     with st.spinner("Analizando texto base..."):
@@ -197,6 +183,7 @@ def display_current_situation_interface(lang_code, nlp_models, t):
                         # Al analizar base, reiniciamos la iteración
                         st.session_state.show_iter = False
 
+            # Columna derecha: Texto iteración
             with col_right:
                 st.markdown("**Texto de iteración**")
                 text_iter = st.text_area(
@@ -210,6 +197,7 @@ def display_current_situation_interface(lang_code, nlp_models, t):
                     with st.spinner("Analizando iteración..."):
                         doc = nlp_models[lang_code](text_iter)
                         metrics = analyze_text_dimensions(doc)
+
                         st.session_state.iter_text = text_iter
                         st.session_state.iter_metrics = metrics
                         st.session_state.show_iter = True
@@ -217,7 +205,6 @@ def display_current_situation_interface(lang_code, nlp_models, t):
         # ---------------------------------------------------------------------
         # Recomendaciones al final en un expander (una sola “fila”)
         # ---------------------------------------------------------------------
-        # Solo mostramos si tenemos iteración
         if st.session_state.show_iter:
             with st.expander("Recomendaciones", expanded=False):
                 reco_list = []
@@ -225,30 +212,35 @@ def display_current_situation_interface(lang_code, nlp_models, t):
                     score = values['normalized_score']
                     target = TEXT_TYPES[text_type]['thresholds'][dimension]['target']
                     if score < target:
-                        suggestions = get_dimension_suggestions(dimension)  # Ejemplo
+                        # Aquí, en lugar de get_dimension_suggestions, unificamos con:
+                        suggestions = suggest_improvement_tools_list(dimension)
                         reco_list.extend(suggestions)
 
                 if reco_list:
-                    # Todas en una sola línea, separadas por " | "
+                    # Todas en una sola línea
                     st.write(" | ".join(reco_list))
                 else:
                     st.info("¡No hay recomendaciones! Todas las métricas superan la meta.")
 
 
 
+
+
+
+
 #Funciones de visualización ##################################
-##############################################################################
-#                  Función para mostrar métricas en una sola fila
-##############################################################################
+############################################################
+# Funciones de visualización para las métricas
+############################################################
+
 def display_metrics_in_one_row(metrics, text_type):
     """
     Muestra las cuatro dimensiones (Vocabulario, Estructura, Cohesión, Claridad)
-    en una sola línea, usando 4 columnas.
+    en una sola línea, usando 4 columnas con ancho uniforme.
     """
     thresholds = TEXT_TYPES[text_type]['thresholds']
     dimensions = ["vocabulary", "structure", "cohesion", "clarity"]
 
-    # Forzamos ancho uniforme de columnas con: st.columns([1,1,1,1])
     col1, col2, col3, col4 = st.columns([1,1,1,1])
     cols = [col1, col2, col3, col4]
 
@@ -257,7 +249,6 @@ def display_metrics_in_one_row(metrics, text_type):
         target = thresholds[dim]['target']
         min_val = thresholds[dim]['min']
 
-        # Determinar estado y color
         if score < min_val:
             status = "⚠️ Por mejorar"
             color = "inverse"
@@ -299,6 +290,7 @@ def display_empty_metrics_row():
             )
 
 
+
 ####################################################################
 
 def display_metrics_analysis(metrics, text_type=None):
@@ -551,16 +543,24 @@ def display_radar_chart(metrics_config, thresholds, baseline_metrics=None):
         st.error("Error al mostrar el gráfico")
 
 #Funciones auxiliares ##################################
-def suggest_improvement_tools(dimension):
-    """Sugiere herramientas basadas en la dimensión"""
+
+
+############################################################
+# Unificamos la lógica de sugerencias en una función
+############################################################
+def suggest_improvement_tools_list(dimension):
+    """
+    Retorna en forma de lista las herramientas sugeridas 
+    basadas en 'ANALYSIS_DIMENSION_MAPPING'.
+    """
     suggestions = []
     for analysis, mapping in ANALYSIS_DIMENSION_MAPPING.items():
-        if dimension in mapping['primary']:
+        # Verificamos si la dimensión está en primary o secondary
+        if dimension in mapping['primary'] or dimension in mapping['secondary']:
             suggestions.extend(mapping['tools'])
-    
-    st.info(f"Herramientas sugeridas para mejorar {dimension}:")
-    for tool in suggestions:
-        st.write(f"- {tool}")
+    # Si no hay nada, al menos retornamos un placeholder
+    return suggestions if suggestions else ["Sin sugerencias específicas."]
+
         
 def prepare_metrics_config(metrics, text_type='student_essay'):
     """