[Proyecto I] EventsManagment

[RDR-03]

Nombre: Omar Alejandro Cuellar Ash
Grupo: C111
**Repositorio: https://github.com/RDR-03/EventsManagement **

Descripción

Herramienta funcional destinada a la organización logística de eventos hoteleros. La aplicación permite a un usuario coordinador gestionar la creación de eventos, administrar un inventario de recursos asociados y, principalmente, prevenir conflictos de disponibilidad horaria entre distintos eventos que comparten recursos limitados.

[github-actions]

Verificación Automática — Proyecto I 🤖

Repositorio analizado: https://github.com/RDR-03/EventsManagement

1. Repositorio

✅ El repositorio es accesible y fue clonado correctamente.

2. Informe (report.md)

⚠️ report.md encontrado pero con solo 1,654 palabras (mínimo: 2,000).

3. Proyecto Python

⚠️ Sin pyproject.toml.
⚠️ Sin requirements.txt.

Archivos Python (14, 1,490 líneas en total):

• core.py
• resources.py
• events.py
• main.py
• core/planification.py
• core/core.py
• core/resources.py
• core/planification2.py
• core/events.py
• core/_init_.py
• UI/home.py
• UI/menus/event_creation.py
• UI/menus/see_inventory.py
• UI/menus/see_schedule.py

4. Punto de Entrada

✅ Punto de entrada detectado: main.py

5. Ejecución

✅ El proyecto se ejecuta.

Traceback (most recent call last):
  File "/tmp/tmpxqqmyqje/repo/main.py", line 1, in <module>
    import streamlit as st
ModuleNotFoundError: No module named 'streamlit'

Resumen

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

4/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/RDR-03/EventsManagement

Qué hace tu proyecto

Construiste un sistema web (Streamlit) para coordinar eventos hoteleros: crear eventos, administrar un inventario de recursos limitados (salones, datashows, técnicos, coffee break…) y, sobre todo, prevenir conflictos de disponibilidad horaria entre eventos que comparten esos recursos. Lo más valioso: modelaste los recursos como un pequeño grafo con dependencias (un datashow necesita pantalla y técnico) y conflictos (Tv vs Datashow no conviven), y un motor que detecta solapamientos temporales. Es ambicioso y está bien resuelto para primer año.

Ejecución

Lo ejecuté de dos formas. Primero levanté la app con streamlit run main.py en modo headless: arrancó sin errores y respondió HTTP 200. Segundo, y esto fue posible gracias a tu buen diseño, ejercité directamente el motor de negocio (core/) con datos reales, sin tocar la interfaz. Probé:

• Evento sin recursos → lo rechaza correctamente.
• Dos eventos solapados pidiendo el mismo datashow (solo hay 2) → detecta el conflicto: "No hay disponibilidad de Datashows en estas fechas". ✅
• Los mismos recursos en días distintos (sin solape) → los acepta. ✅
• Tv + Datashow en un mismo evento → bloqueado por conflicto. ✅
• Pedir 99 salones (hay 5) → rechazado. ✅
• El Asistente de Disponibilidad (find_space): con la franja 10–12 ocupada, sugirió correctamente 12:00–14:00. ✅
• Guardar y volver a cargar desde JSON → el evento se reconstruyó idéntico, con sus fechas. ✅

Todo compila (py_compile OK) y no vi ningún Traceback en los flujos que probé.

Lo que está bien

• Separaste el dominio de la interfaz de verdad. core/ no importa Streamlit en ningún sitio; por eso pude testear el motor solo. Esta es una de las mejores decisiones de arquitectura que se ven a este nivel — mantenla siempre.
• El motor de solapamiento temporal (inicio1 < fin2 and inicio2 < fin1) es correcto y maneja bien el caso difícil (un evento que contiene a otro).
• El modelo de dependencias y conflictos entre recursos es una idea avanzada y la implementaste funcionando.
• La verificación al reducir inventario (avisar si dar de baja un recurso rompería eventos ya agendados, sin descontarlo) muestra cuidado por la integridad de los datos.
• Buena persistencia con JSON e ISO-8601, con round-trip verificado.

Qué mejorar

• Valida que el fin sea posterior al inicio en la lógica, no solo en la UI. Probé valid_event con fin 10:00 e inicio 12:00 y devolvió un evento válido. En la interfaz muestras un error si end < start (event_creation.py:106), pero ese chequeo no pone valid_dates = False, así que no bloquea la confirmación. Añade el guard tanto en valid_event como en la bandera de la UI.
• Declara pandas en pyproject.toml. Lo usas en see_schedule.py pero no está en las dependencias; una instalación limpia fallaría al abrir "Eventos Planificados". Añádelo junto a streamlit.
• Borra el código muerto: valid_datetime (planification.py:111) no se usa en ningún lado (la UI usa widgets de fecha), y core/core.py está vacío. Menos ruido = proyecto más limpio.
• Detalle idiomático: resource_availability modifica resour.in_use sobre el objeto compartido y luego lo resetea. Funciona, pero es frágil (si algo fallara a medias, el inventario quedaría corrupto). Es más seguro calcular en una variable local sin tocar el objeto.
• Menor: las clases van en PascalCase (Resource, Event, Planification), no minúscula; y el archivo core/_init_.py no es un __init__.py real (funciona por importarse a mano, pero confunde).

Sobre el informe

Buen informe: bien estructurado, claro, y supera el mínimo de palabras (la versión actual tiene 2210; el bot vio una previa más corta). Describe fielmente lo que hace el código, y la fórmula de solapamiento de §6.1 coincide exactamente con la implementación. Dos matices para ser precisos: (1) dices haber hecho "un encoder y decoder personalizados" para JSON, pero en realidad usas isoformat()/fromisoformat() dentro de to_dict/from_dict — es una solución correcta, pero no es un encoder personalizado en sentido técnico; (2) llamas a la app "ÁGORA" en la guía de uso, pero ese nombre no aparece en el código (la interfaz se titula "Gestor de Eventos"). Detalles menores.

Buen trabajo. El proyecto tiene un motor correcto y una arquitectura limpia; solo faltan pulir los cabos de robustez para que el interior esté tan cuidado como el diseño. Sigue así.