Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Corrección #21

Open
ferrero-felipe opened this issue Aug 19, 2021 · 0 comments
Open

Corrección #21

ferrero-felipe opened this issue Aug 19, 2021 · 0 comments

Comments

@ferrero-felipe
Copy link

¡Olé Leti! 



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.

Me encanta tu readme y la organización de tu proyecto, super pro! Pero desde ya, no esperábamos nada diferente de ti.

Entiendo la complicación en el webscraping para conseguir todos los datos, tratándose de jugadores en posiciones diferentes y con diferentes características. En ese caso, casi que no hay mucha alternativa, pero asímismo lo has cumplido genial.

Está genial el proyecto, código sin puntas sueltas y todo funcionando bien. Pero no esperábamos nada diferente de ti. ;)

Lo único es que tienes un tacho dicionário en /src/controllers/player_controller.py, en la función filter_players_by_stats. 👀

Si necesitas que todas las keys existan en el dicionário final, crea en un fichero a parte un diccionario con todas las keys y valor None. En seguida con dict(requests.args) cogiemos los valores que existen y puedes hacer un update. 😅

Lo único que te comentaria para avanzar más en el proyecto y en el CRUD con Postgres que se que te ha gustado es mirar como hacer ORM con sqlalchemy. Es el siguiente paso lógico. ;)



Sobre el dashboard en streamlit, nada que decir. Es casi como no-code para ti. Jajaja

Muy buen trabajo!

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant