feat: normalize: clean normalization of text input (#15)

KameniAlexNea commented 1 year ago

J'ai sauvegardé sous le format zip à cause de la limite d'espaces dans un commit (taille du fichier).

À la fin, je vais voir comment réduire tout ça.

billmetangmo commented 1 year ago

Bien reçu, je vais regarder ça

KameniAlexNea commented 1 year ago

Sinon, je pense avoir fait le tour, si tout est bon pour vous aussi, @billmetangmo peut merger

billmetangmo commented 1 year ago

oki les chagements sont validés @KameniAlexNea . J'ai rajouté ton fichier 4.xxxx dans un dossier experiment et j'ai ajouté tes functions de normalisation filter-cameroon.ipynb. J'ai aussi supprimé ton fichier zip aussi , il est pas nécessaire d'après ton commentaire.

Cf https://github.com/mongulu-cm/tchoung-te/pull/49/commits/72ac285654112f566dcb36c86ac7a83cd173ef7a

KameniAlexNea commented 1 year ago

J'ai l'impression que tu n'as pas fait de merge de ma branche

billmetangmo commented 1 year ago

Yes t'as raison. Je suis revenu à l'état précédent. En fait les commits de main que j'ai rajouté se trouvent au-dessus des tiens dans ton historique. Du coup on aura un commit de plus qui sera un commit de merge avec résolution du conflit.

[x] Ce qu'il faudrait faire pour y rémédier c'est que les commits tirés depuis main soient au-dessous des tiens. Comme si tu avais fait refait ta base à partir de main. C'est un rebase.

etape 1: reset ta branche sur ton commit

git checkout alex 
git reset --hard 8b60776e711b512f0ec32679221ed666deffd66b

étape 2: mettre à jour main

git checkout main
git pull --rebase origin main

étape 3: rebase alex sur main

git checkout alex
git rebase master

[x] Ensuite, il faudra modifier ton commit pour ajouter entre parenthèses le numéro de l'issue. ça permet d'avoir sur le commit dans github un lien vers l'issue. C'est l'issue #15 pour toi. git commit --amend -m "an updated commit message"

ça ressemblera donc à ça:

KameniAlexNea commented 1 year ago

Stp @billmetangmo tu peux vérifier si ce que j'ai fait est bien dans la direction attendue ? On n'a pas encore pu discuter du prochain point

billmetangmo commented 1 year ago

Stp @billmetangmo tu peux vérifier si ce que j'ai fait est bien dans la direction attendue ? On n'a pas encore pu discuter du prochain point

Oui @KameniAlexNea c'est dans la bonne direction. T'as fait 70% du boulot. Il reste plus qu'à:

[x] Rajouter ton code de supression des guillements comme décrit ici et là
[x] L'historique n'est pas toujours bon. Mais c'est ma faute, j'ai oublié de te dire qu'il fallait que tu supprimes tous les commits que tu avais pris depuis main avant de rebase.

En fait, il faudrait quitter de ça

Vers ceci

Pour cela,

git checkout -b alex2 
git reset --hard 8b60776e711b512f0ec32679221ed666deffd66b
git checkout main && git pull --rebase origin main && git checkout -
git rebase main
git cherry-pick 80054ed3dd5a4b5d3d2bab414fbf27a8ce91e8a7
git checkout alex
git reset --hard alex2
git push --force

Enfin, il faut modifier tous tes messages de commit de façon à ajouter (#15)

Sinon pour la suite, je te propose de bosser sur #48 puis #50 et #46 ( dans cet ordre). Les 3 sont liés ( les 2 premières étant plutôt rapides à priori) , il s'agit d'utiliser un certain nombre de bonnes pratiques . L'objectif étant de préparer en amont le pouvoir faire la #8 sans trop de soucis.

KameniAlexNea commented 1 year ago

@billmetangmo c'est quand tu veux pour merger

billmetangmo commented 1 year ago

oui @KameniAlexNea tout est bon là. Je merge 👍🏾

mongulu-cm / tchoung-te