silconari / CORE-project-penguins-

First project in CORE school
The Unlicense
0 stars 0 forks source link

Corrección #1

Open ferrero-felipe opened 3 years ago

ferrero-felipe commented 3 years ago

¡Olé Silvia! 



Muy buen trabajo, como ya hemos visto en tu presentación. La corrección será de la siguiente manera, revisaré todo tu código y iré comentando una série de cosas que vaya viendo. Al final, haré apuntes generales.

Cuidado con los prints kilométricos en los jupyter. Jajaja



En /src/to_new_db.py, importas la siguiente línea

from typing import NewType

Pero no te hace falta. Jejeje. Cuidado por dejar muchas cosas sobrando. En ese caso no tiene un impacto grande, pero hay librerías que pesan y tardan un montón. Hay otras cositas parecidas en el resto del código y una u otra repetición de código que podría ser un bucle, como en render_streamlit. Pero nada más!

Te digo, eso son detalles! Has escrito muy buen código en general. Espero que estés orgullosa de tu trabajo. A mi me ha encantado que lo hayas cogido cariño y esa sensación de propriedad sobre él. :rocket:

Mira donde has llegado, Silvia! Has avanzado mucho y aprendido mucho! Sigue así y que se cuide Ale, que le vas a pasar. Jejeje

Nuestro enhorabuena por el trabajo!

image

silconari commented 3 years ago

Hola Felipe!

He quitado más librerías que tenía por ahí cargadas y sin usar 😅 ¿me pones algún ejemplo de algo que podría haber convertido en bucle en render_streamlit? ?

Otra cosa que estuve mirando a última hora, pero la verdad es que no me salió, fue intentar cachear mediante un decorador algunas funciones, para que la carga del dashboard no fuera lenta, que no es el caso porque la base de datos es pequeña, pero quizás lo hubiese necesitado con otro datases.

Como todo mi código del dashboard está metido en la función render_streamlit ¿funcionaría sobre esa o tendría que ir metiendo cada parte del dashboard en pequeñas funciones y añadirles st.cache?