Update modules/studentact/current_situation_interface.py

modules/studentact/current_situation_interface.py

@@ -90,7 +90,6 @@ ANALYSIS_DIMENSION_MAPPING = {
     }
 }
 
-
 ##############################################################################
 #                           FUNCIÓN PRINCIPAL
 ##############################################################################
@@ -98,16 +97,13 @@ def display_current_situation_interface(lang_code, nlp_models, t):
     """
     TAB:
       - Expander con radio para tipo de texto
-
-    Contenedor-1 (2 "filas" con borde):
+    Contenedor-1 (2 filas con borde):
        - Fila 1: Métricas base
        - Fila 2: Métricas iteración
-
     Contenedor-2 (2 columnas):
        - Col1: Texto base
        - Col2: Texto iteración
-
-    Al final, Recomendaciones en una sola línea.
+    Al final, Recomendaciones en un expander (una sola fila).
     """
     # --- Inicializar session_state ---
     if 'base_text' not in st.session_state:
@@ -123,10 +119,10 @@ def display_current_situation_interface(lang_code, nlp_models, t):
     if 'show_iter' not in st.session_state:
         st.session_state.show_iter = False
 
-    # Creamos un TAB
+    # Creamos un tab
     tabs = st.tabs(["Análisis de Texto"])
     with tabs[0]:
-        # 1) Expander con radio
+        # [1] Expander con radio para seleccionar tipo de texto
         with st.expander("Selecciona el tipo de texto", expanded=True):
             text_type = st.radio(
                 "¿Qué tipo de texto quieres analizar?",
@@ -142,32 +138,39 @@ def display_current_situation_interface(lang_code, nlp_models, t):
         # CONTENEDOR-1: Métricas (2 filas con borde)
         # ---------------------------------------------------------------------
         with st.container():
-            # Fila 1: Métricas base
-            if st.session_state.show_base:
-                # Usamos HTML + CSS para enmarcar las métricas en un div con borde
-                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
-                )
-                # ---- Aquí llamamos a la función que muestra las 4 métricas ----
+            # 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)
-                # Cerramos el div
-                st.markdown("</div>", unsafe_allow_html=True)
+            else:
+                display_empty_metrics_row()
+
+            st.markdown("</div>", unsafe_allow_html=True)
 
             # Fila 2: Métricas de iteración
-            if st.session_state.show_iter:
-                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
-                )
+            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
+            )
+
+            if st.session_state.show_iter and st.session_state.iter_metrics:
                 display_metrics_in_one_row(st.session_state.iter_metrics, text_type)
-                st.markdown("</div>", unsafe_allow_html=True)
+            else:
+                display_empty_metrics_row()
+
+            st.markdown("</div>", unsafe_allow_html=True)
 
         # ---------------------------------------------------------------------
         # CONTENEDOR-2: 2 columnas (texto base | texto iteración)
@@ -187,6 +190,7 @@ def display_current_situation_interface(lang_code, nlp_models, t):
                     with st.spinner("Analizando texto base..."):
                         doc = nlp_models[lang_code](text_base)
                         metrics = analyze_text_dimensions(doc)
+
                         st.session_state.base_text = text_base
                         st.session_state.base_metrics = metrics
                         st.session_state.show_base = True
@@ -211,32 +215,27 @@ def display_current_situation_interface(lang_code, nlp_models, t):
                         st.session_state.show_iter = True
 
         # ---------------------------------------------------------------------
-        # Recomendaciones al final (una sola línea, por ejemplo)
+        # Recomendaciones al final en un expander (una sola “fila”)
         # ---------------------------------------------------------------------
         # Solo mostramos si tenemos iteración
         if st.session_state.show_iter:
-            # Podríamos mostrar todo en una misma línea usando st.columns(1)
-            # o simplemente un st.container(). Lo muestro en un solo contenedor:
-            with st.container():
-                st.markdown("**Recomendaciones**")
-                # Si las quieres en una sola línea, podrías hacer algo como:
-                # rec_col = st.columns(1)[0]   # Solo 1 columna
-                # con rec_col: ...
-                # Pero en general, un for con st.write estará apilado verticalmente.
-
-                # Ejemplo de iterar dimensiones
-                any_reco = False
+            with st.expander("Recomendaciones", expanded=False):
+                reco_list = []
                 for dimension, values in st.session_state.iter_metrics.items():
                     score = values['normalized_score']
                     target = TEXT_TYPES[text_type]['thresholds'][dimension]['target']
                     if score < target:
-                        suggest_improvement_tools(dimension)
-                        any_reco = True
+                        suggestions = get_dimension_suggestions(dimension)  # Ejemplo
+                        reco_list.extend(suggestions)
 
-                if not any_reco:
+                if reco_list:
+                    # Todas en una sola línea, separadas por " | "
+                    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
@@ -249,7 +248,8 @@ def display_metrics_in_one_row(metrics, text_type):
     thresholds = TEXT_TYPES[text_type]['thresholds']
     dimensions = ["vocabulary", "structure", "cohesion", "clarity"]
 
-    col1, col2, col3, col4 = st.columns(4)
+    # 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]
 
     for dim, col in zip(dimensions, cols):
@@ -278,6 +278,27 @@ def display_metrics_in_one_row(metrics, text_type):
             )
 
 
+# -------------------------------------------------------------------------
+# Función que muestra una fila de 4 columnas “vacías”
+# -------------------------------------------------------------------------
+def display_empty_metrics_row():
+    """
+    Muestra una fila de 4 columnas vacías (Vocabulario, Estructura, Cohesión, Claridad).
+    Cada columna se dibuja con st.metric en blanco (“-”).
+    """
+    empty_cols = st.columns([1,1,1,1])
+    labels = ["Vocabulario", "Estructura", "Cohesión", "Claridad"]
+
+    for col, lbl in zip(empty_cols, labels):
+        with col:
+            col.metric(
+                label=lbl,
+                value="-",
+                delta="",
+                border=True
+            )
+
+
 ####################################################################
 
 def display_metrics_analysis(metrics, text_type=None):