No calia que fessis PR perquè el repo es vostre, però sí calia fer una issue. La necessito per saber que heu entregat el projecte, per poder veure el codi i per poder donar-vos feedback. És més important la issue dels projectes que las PR dels labs.
Però no passa res! Igualment trobada.
Et comento:
L'organització del repo 📚:
Esta bé. Tens la src/, images/ (i a més la tens plena d'imatges) el README.md i els notebooks que tenen significat.
Tens el .gitignore, o sigui qeu també perfecte. I a més has ficat l'arxiu perquè no es pengi.
No tens el codi modularitzat dins dels cleaning.py & visualizing.py, tot i que no és necessari encara. Pel proper projecte si que ho demanarem.
L'arxiu de new_sharks.csvhauria d'anar a la carpeta de data, que no hi ha (la tenies ignorada). Sí que pots tenir data, però no el "attacks.csv" original, per la resta de dades (netes), sí que pots.
Tens visualization_1.ipynb y visualization.ipynb. Entenc que vas fer una còpia per fer proves, però un d'ells t'ha deixat de funcionar. Si el notebook està malament, deixa'l fora del repo. Més si el nom és confús.
El README.md 📝: Esta super bé. Has recordat també posar l'enllaç al canva 👌 . L'única cosa que canviaria, és: afegir els plots (que pel que veig ho has fet però no s'han renderitzat: la sintaxi és: ) i posar bullet points al markdown per fer llistes. Per la resta: perfecte. Senzill i explicatiu.
El codi 🐍:
Només hi ha un parell de coses que et diria per canviar:
def activity_standarize(str_):
if pd.isna(str_):
return "others"
str_lower = str_.lower()
if 'swim' in str_lower:
return "swimming"
if any(activity in str_lower for activity in ['bath', 'float', 'crouch', 'dangl',]):
return "bathing"
if any(activity in str_lower for activity in ['fish', 'hunt', 'net']):
return "fishing"
if any(activity in str_lower for activity in ['natural']):
return "natural disasters"
if any(activity in str_lower for activity in ['surf', 'board', 'paddel', 'tread']):
return "surfing"
if any(activity in str_lower for activity in ['air', 'parachut']):
return "air accident"
if any(activity in str_lower for activity in ['boat', 'ship', 'wreck', 'yacht', 'submarine','sink']):
return "sailing"
if 'div' in str_lower:
return "diving"
return "others"
aquí podries haver fet servir regex o haver fet servir un diccionari: {"surfing": ["surf", "surfing board", "sorf"]}, etc.
Has fet servir funcions, o sigui que: 👏👏. El codi per fer el drop, rename, etc també pots ficar-ho en una mateixa funció. I els plots!
Per les visualiztacions 📊:
Super netes i clares. Només el que et vaig dir de l'ordre de l'eix: morning / afternoon / night.
El projecte esta bé! Només coses petites per millorar-lo una mica, però congrats! Pel proper, us demanarem modularitzar. Ànims Pol, ho tens!! 💪
Hola @polsanchez1997! 🙋♂️
No calia que fessis PR perquè el repo es vostre, però sí calia fer una issue. La necessito per saber que heu entregat el projecte, per poder veure el codi i per poder donar-vos feedback. És més important la issue dels projectes que las PR dels labs.
Però no passa res! Igualment trobada.
Et comento:
L'organització del repo 📚:
src/,images/(i a més la tens plena d'imatges) elREADME.mdi els notebooks que tenen significat..gitignore, o sigui qeu també perfecte. I a més has ficat l'arxiu perquè no es pengi.new_sharks.csvhauria d'anar a la carpeta de data, que no hi ha (la tenies ignorada). Sí que pots tenir data, però no el "attacks.csv" original, per la resta de dades (netes), sí que pots.El README.md 📝: Esta super bé. Has recordat també posar l'enllaç al canva 👌 . L'única cosa que canviaria, és: afegir els plots (que pel que veig ho has fet però no s'han renderitzat: la sintaxi és:
) i posar bullet points al markdown per fer llistes. Per la resta: perfecte. Senzill i explicatiu.El codi 🐍:
aquí podries haver fet servir regex o haver fet servir un diccionari:
{"surfing": ["surf", "surfing board", "sorf"]}, etc.Per les visualiztacions 📊:
El projecte esta bé! Només coses petites per millorar-lo una mica, però congrats! Pel proper, us demanarem modularitzar. Ànims Pol, ho tens!! 💪