ta-data-bcn
commented
1 year ago Buenaaass Andrés! 👋🏾
Aquí te va un poco de feedback del penúltimo proyecto.
Respecto a la organización del repositorio 📚
- No subiste el .env, pero se te coló un .DS_Store al repo.
- Aparte de eso la organización del repo está perfeeecta.
El README.md ✍🏽
- Está súper bien como siempre, muy bien explicado y bien por poner screenshots de las gráficas de Tableau y aparte el link para poder verlas de manera interactiva.
- También muy bien por poner los puntos a mejorar del proyecto.
El código 👨🏽💻
- El pipeline funciona peeerfecto.
- Siempre te lo digo, pero de verdad se agradece leer un código tan bien organizado y tan bien comentado como el tuyo 😊
- Por lo que vi, no hiciste ningún scrapping para sacar la info de Wikipedia, ¿lo hiciste a mano? 😨 podrías haber hecho un scrapping y un merge con la df para agregarle los datos que querías, ¿no?
- La verdad no tengo nada malo que decirte. Tu código está muy bien, subiste todo a SQL, hiciste queries (pocas, pero efectivas) y procesaste muy bien los archivos de texto para convertirlos en un dataframe. Sin duda hiciste lo que el proyecto pedía y lo hiciste muy bien.
- Si quieres en algún futuro mejorar algo, podrías ver qué está pasando con los datos del episodio 2.
Visualización (mapas) 🗺
- Muy bien por hacer una story con todos los dashboards.
- En la gráfica de las líneas por personaje por episodio, podrías haber quitado el episodio que sabes que está mal, para que se puedan comparar mejor los que sí están bien, y dejar una nota diciendo que omitiste ese episodio.
- Puedes quitar el "tag" y el "Written by" de la última gráfica, y juntar las 2 Rachel's que tienes.
- Quitando lo anterior, están súper bien las visualizaciones. En especial la última en la que le puedes ir cambiando la temporada
Está increíble el proyecto Andrés, sin duda estoy un poco biased porque me encanta el tema, pero te quedó muy bien 👏🏾 así que muchas felicidades y ánimo en el proyecto final. No me cabe duda que seguro haces algo increíble :)
https://github.com/castrokike/project-IV-sql-tableau