colav / impactu

Colav Impactu Issues and Documentation
BSD 3-Clause "New" or "Revised" License
0 stars 1 forks source link

Revisar informe de mediciones de tiempos de respuesta de los indicadores #189

Closed luismvargasg closed 3 weeks ago

luismvargasg commented 3 months ago

El objetivo de este issue es revisar el informe que contiene las mediciones detalladas de los tiempos de respuesta de los indicadores en el dashboard de los distintos perfiles de entidades en ImpactU, con el fin de generar una lista de acciones prioritarias para mejorar la eficiencia y reducir los tiempos de carga.

Revisar descripción en el Milestone y el siguiente informe: https://docs.google.com/spreadsheets/d/1jU5hCV3W9OUJ2XDPdg10VE7JWIIxoSPoOHog5dO96-k/edit?usp=sharing

luismvargasg commented 2 months ago

Algo pasó con las correcciones hechas en las gráficas que volvieron con los tiempos de respuesta anteriores: image Podría ser un tema de índices @omazapa ? O quizás se perdió el cambio/commit donde se arreglaban esas respuestas @joselomanuelo ?

gabrielvelezcuartas commented 2 months ago

Creo que el el problema de estos tiempos:

https://github-production-user-asset-6210df.s3.amazonaws.com/60363752/358110775-4a6eef1a-f3a9-4914-9233-7477b6f74ee0.png?X-Amz-Algorithm=AWS4-HMAC-SHA256&X-Amz-Credential=AKIAVCODYLSA53PQK4ZA%2F20240815%2Fus-east-1%2Fs3%2Faws4_request&X-Amz-Date=20240815T134238Z&X-Amz-Expires=300&X-Amz-Signature=4f9b5d91000ae41749d518c416385138847e5a5a9070fc9c5b50403dc3a33301&X-Amz-SignedHeaders=host&actor_id=73314075&key_id=0&repo_id=772097425

Se puede arreglar como proponia, disminuyendo el número de grupos o subunidades académicas que se presenten en el treemap. No se si se logra, pero la idea seria que nos fueran más de 20 grupos o subunidades academicas por pantalla como había propuesto en las correcciones del dashboard. No es util que se carguen todos los grupos en un solo treemap, pues son más de 200, eso es ilegible.

luismvargasg commented 2 months ago

De acuerdo profe @gabrielvelezcuartas, por ahora estamos revisando a qué se debe el retroceso en el código porque ya estaba funcionando, una vez tengamos corregido eso desde back ya entramos a realizar esos cambios, es correcto, mostrar 200 campos en un treemap no es para nada funcional.

joselomanuelo commented 2 months ago

Se implementa la nueva arquitectura en los plots, se evidencia tiempos rápidos en respuesta de todos los plots usando entidades aleatorias:

Image

Con algunas entidades el tiempo puede aumentar sensiblemente, por lo tanto se deja en pausa la revisión de las consultas a base de datos hasta tener la nueva lista de plots necesarios por parte de frontend.

luismvargasg commented 3 weeks ago

Se cierra este issue y se le da seguimiento en https://github.com/colav/impactu/issues/279