Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Minors y titulos faltantes #366

Merged
merged 28 commits into from
Sep 27, 2023
Merged

Minors y titulos faltantes #366

merged 28 commits into from
Sep 27, 2023

Conversation

kovaxis
Copy link
Contributor

@kovaxis kovaxis commented Sep 25, 2023

Limpie el codigo del scraper de minors y titulos, y lo movi al planner mismo. Antes el flujo de datos era algo:

        SIDING -> Consolidacion -> Mock SIDING -> Ingesta planner -> Parches planner
PDFs -> Scrape ->     manual
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
       offline, al momento de desarrollo          online, cada vez que corre planner

Ahora es mas como:

SIDING -> Mock SIDING -> Ingesta -> Parches y transformaciones planner
       PDFs -> Scrape -> planner
^^^^^^^^^^^^^^^^^^^^^    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
       offline                 online, cada vez que corre planner

Hay dos razones por la que el mock de SIDING sigue siendo offline (en lugar de descargar los planes desde SIDING al momento de correr planner):

  • Permite que contribuidores sin acceso al webservice de SIDING puedan correr planner localmente con funcionalidad completa.
  • Evita que cambios inesperados en el webservice de SIDING rompan planner. De hecho, actualmente es necesario este parche porque el webservice sigue roto desde principio de año y no entrega versiones de los curriculums.

Para los planes más complejos, agregue una fuente adicional de planes curriculares, que es simplemente hardcodear los planes en un formato más flexible.

¿Qué se implementa?

  • Un nuevo sistema de ingesta de datos curriculares.
  • Agregar todos los minors y titulos faltantes :D (completa en gran parte Faltan majors, minors y títulos #307).
  • Cachear los datos curriculares en memoria local.
  • Hacer que Cyear sea directamente una union de literales en lugar de una clase dedicada, y los cambios asociados en el frontend.

¿Qué falta?

Solo faltan 5 majors, que SIDING no entrega correctamente:

  • (M235) Major en Ingeniería, Diseño e Innovación - Track en Ingeniería Vs.02 (major solo para C2020)
  • (M143) Major en Ingeniería Física - Track Ingeniería Vs.01
  • (M149) Major en Ingeniería Física - Track Física Vs.01
  • (M188) Major en Ingeniería Civil - Track Ingeniería Geotécnica Vs.02 (C2020 esta bien, falta solo para C2022)
  • (M148) Major en Ingeniería Biomédica - Track Premedicina Vs.02 (C2020 esta bien, falta solo para C2022)

SIDING tiene que agregar estos majors.

@kovaxis kovaxis requested a review from fagiannoni September 25, 2023 15:14
Copy link
Contributor

@fagiannoni fagiannoni left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 todo excelente! Tremendo PR 🚀

@fagiannoni fagiannoni merged commit 1438583 into dev Sep 27, 2023
@fagiannoni fagiannoni deleted the missing-mallas branch September 27, 2023 22:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants