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

Green view index #6

Closed PyMap closed 1 year ago

PyMap commented 1 year ago

Qué hace este PR:

  1. Reordena el sidebar en ´home´ y expanders para templates de micro o macro modelling.
  2. Incorpora el directorio ´streets_network´. La idea es agrupar ahí todas las operaciones que hagamos sobre calles.
  3. Actualiza el Readme con instrucciones para instalar ´pymeanshift´
  4. Renombra a ´main.py´ el antiguo ´urban_modeller.py´
  5. Incorpora el template de micromodelling ´streets_greenery.py´. Esta es una version acabada de la implementación de un modelo para la resolución de un problema urbano. A su vez, es un ejemplo de cómo ajustar un template a la botonera de 4 secciones: simulation frame, Explore results, Explore zones, Explore impact
  6. Actualiza datasources.py con las bases de datos utilizadas

Qué no hace este PR:

  1. El sidebar todavía no resuelve bien cómo salir y entrar del home. Este debería mejorarse cuando agreguemos las secciones que faltan (tanto futuros templates de micromodelling como ABOUT US). Se podría evaluar si un diseño multipágina es algo que a futuro nos va a hacer la vida más fácil.
  2. No resuelve la instalación de pymeanshift via requirements. A futuro se podría reemplazar por otro algoritmo de segmentación. De momento es el que tenemos.
  3. No centraliza el consumo de bases de datos. Si alguien no tiene una mejor alternativa, podríamos pensar en usar un model_data.h5
NMonnerSans commented 1 year ago

Antes de empezar, lo de multipágina me parece genial. Es lo que yo decía el otro día (lo llamé routing)

NMonnerSans commented 1 year ago

The PR to the PR (the part after Fede's as of today last comment), does the following stuff:

PyMap commented 1 year ago

I really liked how street greenery and green surfaces templates are designed as subclasses of the Dashboard rules. This gives us a clear structure which I think it is not only useful for safety purposes but also to write the upcoming templates. I also liked the multipages configuration. The widgets are very useful too.

Thanks for your wonderful work @NMonnerSans!!

PyMap commented 1 year ago

Let´s merge it!