ta-data-bcn
commented
10 months ago Holaa Miki! 🚀
¡Muchas felicidades por tu proyecto! Antes de nada quiero decirte que es un proyectazo, se nota que le has metido horas y que te interesa el tema. A mi me ha parecido súper interesante de leer y definitivamente lo podrás usar en tu portfolio a la hora de buscar trabajo o enseñar lo que sabes hacer.
Un poco de feedback - Respecto a la organización del repositorio 📚
- El repo está muy bien organizado, ojo con los DS.Store que se te han colado, te recomiendo que siempre al crear un repo lo hagas con un .gitignore que ya te da hecho gthub, así estos DS.store se te van a ignorar automáticamente.
El README.md 📖
- Que te voy a decir… Es de los readmes mejor explicados que he visto hasta ahora, sigue un hilo muy interesante y no se hace pesado ni largo.
- Cuando hablas de tus findings de SQL estaría bien saber de qué findings se trata, mostrar un poco más los resultados de las queries.
- Por lo demás SUPER TOP!!!!!!!!
- Ya he visto que has cambiado lo de Saint Emilion ;)
Code,SQL, Tableau 👩💻
- Las Queries de SQL las puedo ver en el archivo .sql pero no puedo ver los resultados.
- Podrías hacer un notebook de jupyter conectado a sql para que también pueda ver los resultados de las queries la persona que lo lea.
- El tableau está brutal también! Te felicito. Lo único que te puedo aconsejar es igual poner algún filtro para que sea algo más interactivo, digamos para poder hacer focus en una región o año y que el tableau modifique las tablas acordemente.
Por lo demás super bien Miki!!!!!! Suerte en el Final!!!!!!!!!!!!!
https://github.com/Miquelpg6/project_4