[Proyecto I]

[GuillermoRR06]

Nombre: Guillermo Rodriguez Rodriguez
Grupo: C121
Repositorio: https://github.com/GuillermoRR06/Management-System--CC-.git

Descripción

Gestor inteligente de eventos de un cine-teatro

[github-actions]

Verificación Automática — Proyecto I 🤖

Repositorio analizado: https://github.com/GuillermoRR06/Management-System--CC-.git

1. Repositorio

✅ El repositorio es accesible y fue clonado correctamente.

2. Informe (report.md)

❌ No se encontró report.md en el repositorio.

3. Proyecto Python

⚠️ Sin pyproject.toml.
✅ requirements.txt encontrado.
Dependencias declaradas: altair, attrs, blinker, cachetools, certifi, charset-normalizer, click, colorama, gitdb, GitPython

Archivos Python (11, 1,065 líneas en total):

• App/Inicio.py
• App/Functions/__init__.py
• App/Functions/RevResources.py
• App/Functions/EventsFuncs.py
• App/Functions/AuxFuncs.py
• App/Functions/Save_Data.py
• App/pages/➕ Agregar Eventos.py
• App/pages/📅 Lista de Eventos.py
• App/pages/📋 Informacion de Recursos.py
• App/Intellisense/__init__.py
• App/Intellisense/Funcs.py

4. Punto de Entrada

⚠️ No se encontró un punto de entrada claro (main.py, app.py, etc.).

5. Ejecución

⚠️ Sin punto de entrada identificado — no se intentó ejecutar.

Resumen

• ✅ Repositorio accesible
• ❌ report.md con ≥2000 palabras
• ✅ Archivos Python
• ✅ Configuración de dependencias
• ❌ Punto de entrada
• ❌ Ejecución

3/6 verificaciones pasadas.

---

Verificación automática — los resultados son orientativos. El profesor revisará en detalle.

[apiad]

🔍 Revisión de Código — Claude Code

Repositorio: https://github.com/GuillermoRR06/Management-System--CC-.git

Qué hace tu proyecto

Construiste un gestor de eventos para tu cine-teatro "La Cuarta Pared" con Streamlit:
agregar proyecciones, obras y conciertos validando colisiones de salas y personal, ver la
lista de eventos por día o por 30 días, ver detalles, eliminar, y consultar los recursos.
Lo más valioso es la arquitectura: separaste con mucha claridad la UI (App/pages/), la
lógica de negocio (App/Functions/), la persistencia (Save_Data.py) y el agendador
automático (Intellisense/Funcs.py). Y usaste estructuras no triviales —búsqueda binaria y
un merge sort propio sobre el calendario— que hablan de ambición.

Ejecución

Como es una app web (no de consola), la evalué de dos formas: corriendo directamente tu
lógica de negocio con los datos reales del repo, y arrancando la GUI en modo headless.
La lógica núcleo funciona bien: agregué 3 películas de 10 a 12 usando 2 técnicos de sonido
cada una y Disponibility dejó sonido=0 y esas 3 salas ocupadas, correcto; el concierto
bloquea correctamente su franja (y el colchón de 30 min); las cuotas por asistencia validan
bien.

Ahora lo importante: sobre un clon recién hecho, la app no arranca en Linux. Al cargar
los datos, Save_Data.GetData() falla con:

FileNotFoundError: [Errno 2] No such file or directory: 'Data\data.json'

Es por la ruta con backslash de Windows ("Data\data.json"). En Windows funciona; fuera de
Windows, \d no es una carpeta sino un nombre de archivo con backslash.

Lo que está bien

• Separación de responsabilidades clara y madura; nombres descriptivos y docstrings en casi todo.
• El flag k en tus validaciones (RevResources.py) para reusarlas con o sin mensajes de error
  en la UI y en el agendador silencioso: patrón inteligente.
• Reglas de dominio bien implementadas y verificadas al ejecutar: escenario modular para
  teatro/concierto, exclusividad del concierto (Check_MC/Check_Evs), cuota de limpieza y
  seguridad por asistencia, y el colchón de 30 min tras cada evento.
• BS_Date (búsqueda binaria) y Sort_Dates (merge sort) bien escritos.
• Informe de 18 páginas, cuidado, honesto y con voz propia.

Qué mejorar

• Rutas portables (lo más importante). Cambia "Data\data.json" por "Data/data.json" o
  os.path.join("Data", "data.json") en Save_Data.py; lo mismo en las imágenes de
  Inicio.py ("App\Gallery\...") y en el st.switch_page("pages\..."). Con eso la app arranca
  en cualquier sistema. Es un arreglo de una línea por sitio y el de mayor impacto.
• Orden cronológico real en AddEvent (EventsFuncs.py:41-50). Al probarlo, agregar un
  evento de las 08:00 después de otros de las 10:00 lo dejó al final de la lista, no ordenado
  ([10:00, 10:00, 10:00, 08:00]). La condición if hora < tm_Init no ordena de verdad, y el
  elif hora == tm_Init and hora.minute < tm_Init.minute nunca se cumple (si dos time son
  iguales, sus minutos ya lo son). Se ve bien solo porque la lista se muestra con reversed().
• FindNewDay (Intellisense/Funcs.py:155). El chequeo len(evens[j]) < 6 no mide cuántos
  eventos tiene el día: mide cuántas claves tiene el diccionario del día (siempre 3), así que
  es siempre verdadero. Además j = BS_Date(...) da -1 para días futuros aún vacíos, y evens[-1]
  no es ese día. Resultado: sugiere los próximos 3 días sin comprobar si están libres.
• Un solo data.json como fuente de verdad. Disponibility (RevResources.py:22-29) hardcodea
  las cantidades de personal (6, 6, 8...) con claves sin tildes, pero tu JSON las guarda con
  tildes en Recursos.humanos. Cambiar el inventario en el JSON no cambiaría la validación. Lee
  esas cantidades del JSON para que todo dependa de una sola fuente.
• Detalle: "Events" not in st.session_state (Inicio.py:6) es un typo — guardas "Eventos". La
  condición queda siempre verdadera; funciona por casualidad. Corrige a "Eventos".
• Menor: tus f-strings con comillas dobles anidadas (f"...{event["descripcion"]}...") solo
  compilan en Python 3.12+. Para que corra en versiones anteriores, usa comillas simples dentro.

Sobre el informe

Muy bueno: 18 páginas, bien estructurado, honesto (no inventa pruebas automáticas — los resultados
son capturas de la app en uso). Un matiz: el resumen la llama "robusta" y dice cumplir "todos" los
objetivos, lo cual es una ligera sobreestimación dados los tres bugs de arriba (sobre todo que no
arranca fuera de Windows). Punto a favor de tu transparencia: el propio informe muestra el código con
el guard "Events", así que documentaste fielmente lo que hiciste.

¡Buen trabajo, Guillermo! La base de diseño es de las sólidas; con esos arreglos —empezando por las
rutas— quedaría redondo.