CEEU-lab / city_modeller

Urban dynamics performance assessment with data-driven modelling templates
GNU General Public License v3.0
0 stars 0 forks source link

Public spaces cleanup #3

Closed NMonnerSans closed 1 year ago

NMonnerSans commented 1 year ago

Lo que hace este PR:

Lo que le falta, pero pienso agregarle:

NMonnerSans commented 1 year ago

Lo que le falta, pero pienso agregarle:

  • La función de generar la bbox tiene copypaste que me gustaría simplificar, mañana le pego una mirada.
  • La creación del dashboard es una sola función enorme y monolítica. Me gustaría abstraer cada container en una funcioncita, para facilitar la debuggeación y el mantenimiento.
  • Quizás el código ya tiene un tamaño digno de meter tests.

Metí varios tests y mejoré la función de la bbox, bastante. Es un poco más rápida porque usa tuplas en vez de crear tantos Points que después descompone en tuplas. Igual obviamente no es eso lo que hace lento el streamlit, pero un par de ms recortamos, y unas líneas de código también. @fabridiana te dejo esto updateado para que lo mires si gustás. Querría hacer más cosas, pero estamos en situación de demo con esto. Si la performance ya alcanza, mejor mirar lo de las geometries.