carlajsv
commented
1 year ago Holii Georgina👋🏾
👇🏾
Organization of your files 🧹
Tu git.ignore funciona perfecto, gracias por no subir attacks.csv Muy bien al haber hecho una carpeta de data con los data sets limpios.
README.md ✍🏽
El readme esta muy bien explicado y simplificado, sin embargo, te sugeriria que para proximos proyectos te ayudaras con imagenes o llos graficos que sacaste parta entender mas alla de lo que querias sacar, es decir, incluir en el readme graficos que me ayuden a mi o a cualquier lector de donde salen las conclusiones que sacaste y cual fue el procedimiento que usaste.
Dentro de todo, tu README esta muy bien explicado y se entiende en lineas generales lo que hiciste.
The code < > 👨🏽💻
Clean.ipnyb
Muy buena practica importar las librerias al principio, es algo que se debe mantener, sobretodo cuando tenemos archivos de codigo muy grandes.
Una sugerencia, cuando estamos trabajando en proyectos, muchas veces tenemos la necesidad de imprimir el df para ver como quedó o simplemente entender si lo que estamos haciendo se hizo correctamente, sin embargo, a la hora de entregar un proyecto yo no dejaría las impresiones, ya que para unos como lector se hace visualmente engorroso ver tantas tablas iguales.
Para hacer referencia a procedimientos que seguiste en lineas de codigo, te recomendaria usar los comentarios que empiezan con el # (los que salen en color azul), las celdas de markdown las dejaria para titulos, subtitulos y explicaciones mas generales.
Muchas gracias por utiliza titulos que ayuden a a entender de donde a donde van los procedimientos que seguiste.
Cuando subas el proyecto, o mejor dicho, cuando hagas push de tu notebook, recuerda cerrar las celdas que tienen listas super largas y quizas no aporten tanto al lector, te doy un ejemplo de esto:
Debajo del titulo de Columnas nuevas de: Months, hemisferio y estaciones del año en la 4ta celda siguiente hiciste un sorted esta celta impresa es muy larga y dificil de pasar, con cerrarla antes de subir el notebook esta guay.
Analysis.ipyb
Con respecto al notebook de analisis, esta muy ordenado.
Con respecto a la primera hipotesis, si estaría guay saber qué te llevé a decidir sobre quedarte solo con dos países, por qué los otros no? y cuántos habían o encontraste en total? el contexto en estos casos es super importante.
Viz 🎨
Muy buen uso de los pie charts. Muchas veces son muy mal usados, sin embargo, con dos porcentajes esta mas que bien, tal y como lo hiciste, asi que bravo.
Recuerda que las visualizaciones deben tener datos representativos y explicar por qué decidiste quedarte solo con un porcentaje de datos y cuál es ese porcentaje.
Recuerda tomar en cuenta el tema del proyector y el tamaño de las fuentes para el proximo proyecto.
En general me gusto mucho como enfocaste el proyecto, muy ordenado, no te enrollaste tanto en las funciones que usaste y tu codigo esta muy limpio.
Bravo! 👏👏👏
https://github.com/georginamanyanic/Project-1.git