search

surfriderfoundationeurope / etl

ETL (Extract Transform Load) Data Management process
MIT License
2 stars 0 forks source link

Dev/code review #6

Closed bertrandlalo closed 4 years ago

bertrandlalo commented 4 years ago

Hello la team ETL :) @cl3m3nt , @rcourivaud

J'ai fait pas quelques refactoring, en appliquant es corrections détaillées dans cette issue.

Pour résumer :

Changement de forme

Réorganisation du repo

J'ai essayé de faire du tri dans tous les codes, et ce que je propose comme orga du repo, c'est :

PEP 8 et doc-string

Pandas

Misc

Optimisations

Nouvelles features

Correction du trashGPS

Accepte les entrées de type Smartphone vidéos et Manuelles

L'hypothèse faite ici, c'est qu'il y a 4 types de source d'acquisition possibles et que les fichiers GPX associés aux médias portent le même nom (@AntoineBruge, ok?) :

NB1: La sortie de la version manuelle est légèrement différente:

NB2: Les labels des données manuelles sont différents : {'Objet vie courante', 'Bouteille boisson', 'Peche et chasse', 'Autre dechet', 'Industriel ou construction', 'Emballage alimentaire', 'Dechet agricole', 'Autres dechets +10'}

Pour tester ça en local, vous pouvez faire (après avoir activé l'environnement, cf README.rst) :

Pouvoir travailler 100% en local (sans se connecter à Azure & Postgre)

En fait, quand on veut tester ou développer, c'est pratique de ne pas être dépendant de Postgre et de Azure. Donc tu verras, j'ai essayé de documenter, mais dans la commande pour lancer l'ETL, on peut choisir de travailler avec des données locales plutôt que de télécharger depuis Azure (on pourra comme ça imaginer à l'avenir télécharger depuis d'autres sources). Pareil pour la sortie, on peut choisir de sauver le résultat en CSV localement plutôt que de l'insérer dans la base de données.

Todo, next steps

Qu'en dites vous ?

++

AntoineBruge commented 4 years ago

Salut tout le monde,

Merci Raph pour ton mail. Tu as fait un travail monstre ! Pour répondre à tes questions : il n’y a que 3 sources d’acquisition possible ->

On en parle demain 😉 Bonne aprem Antoine

From: Raphaëlle Bertrand-Lalo notifications@github.com Sent: vendredi 1 mai 2020 16:48 To: surfriderfoundationeurope/etl etl@noreply.github.com Cc: Surfrider Antoine Bruge abruge@surfrider.eu; Mention mention@noreply.github.com Subject: [surfriderfoundationeurope/etl] Dev/code review (#6)

Hello la team ETL :) @cl3m3nthttps://github.com/cl3m3nt , @rcourivaudhttps://github.com/rcourivaud

J'ai fait pas quelques refactoring, en appliquant es corrections détaillées dans cette issuehttps://github.com/surfriderfoundationeurope/etl/issues/5.

Pour résumer :

Changement de forme Réorganisation du repo

J'ai essayé de faire du tri dans tous les codes, et ce que je propose comme orga du repo, c'est :

PEP 8 et doc-string

Pandas

· latitude longitude elevation

·

· 2018-01-24 19:27:58+00:00 33.126515 -117.327168 -17.228

·

· 2018-01-24 19:27:59+00:00 33.126543 -117.327153 -18.199

·

· 2018-01-24 19:28:12+00:00 33.126616 -117.325690 -18.440

·

· 2018-01-24 19:28:13+00:00 33.126540 -117.327072 -19.161

·

· 2018-01-24 19:28:14+00:00 33.126539 -117.327218 -18.472

·

Misc

[Screenshot 2020-05-01 at 16 25 37]https://user-images.githubusercontent.com/22340670/80812514-6b693b00-8bc8-11ea-9354-c643d7d80acb.png source build à compléter ! Optimisations

Nouvelles features Correction du trashGPS

Accepte les entrées de type Smartphone vidéos et Manuelles

L'hypothèse faite ici, c'est qu'il y a 4 types de source d'acquisition possibles et que les fichiers GPX associés aux médias portent le même nom (@AntoineBrugehttps://github.com/AntoineBruge, ok?) :

NB: La sortie de la version manuelle est légèrement différente:

Pouvoir travailler 100% en ligne

En fait, quand on veut tester ou développer, c'est pratique de ne pas être dépendant de Postgre et de Azure. Donc tu verras, j'ai essayé de documenter, mais dans la commande pour lancer l'ETL, on peut choisir de travailler avec des données locales plutôt que de télécharger depuis Azure (on pourra comme ça imaginer à l'avenir télécharger depuis d'autres sources). Pareil pour la sortie, on peut choisir de sauver le résultat en CSV localement plutôt que de l'insérer dans la base de données.

Todo, next steps

Qu'en dites vous ?

++

You can view, comment on, or merge this pull request online at:

https://github.com/surfriderfoundationeurope/etl/pull/6

Commit Summary

File Changes

Patch Links:

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHubhttps://github.com/surfriderfoundationeurope/etl/pull/6, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ALB2U2Z2525GTAWNTWLWHTTRPLOKNANCNFSM4MXEQ45A.

cl3m3nt commented 4 years ago

Bonjour tout le monde,

merci pour les infos complémentaires Antoine et merci Raphaelle pour toutes les propositions ! On parle de tout ça tout à l'heure, on aura pas mal de choses voir ensemble j'ai l'impression :) A tout à l'heure.

Clement

On Mon, May 4, 2020 at 5:41 PM Antoine Bruge notifications@github.com wrote:

Salut tout le monde,

Merci Raph pour ton mail. Tu as fait un travail monstre ! Pour répondre à tes questions : il n’y a que 3 sources d’acquisition possible ->

  • gopro: fichier foo.mp4 (données GPS intégrées au .mp4)
  • smartphone video depuis l’app Plastic Origins (en développement) : fichier foo.mov + fichier foo.gpx
  • Données manuelle depuis l’app Plastic Origins : fichier foo.gpx (incluant la trace GPS + des waypoints correspondant aux déchets) Les données GPS seront toutes stockées dans le format GPX. C’est le format qui nous semble être (merci Clément) le plus standard et facile à manier.

On en parle demain 😉 Bonne aprem Antoine

From: Raphaëlle Bertrand-Lalo notifications@github.com Sent: vendredi 1 mai 2020 16:48 To: surfriderfoundationeurope/etl etl@noreply.github.com Cc: Surfrider Antoine Bruge abruge@surfrider.eu; Mention < mention@noreply.github.com> Subject: [surfriderfoundationeurope/etl] Dev/code review (#6)

Hello la team ETL :) @cl3m3nthttps://github.com/cl3m3nt , @rcourivaud< https://github.com/rcourivaud>

J'ai fait pas quelques refactoring, en appliquant es corrections détaillées dans cette issue< https://github.com/surfriderfoundationeurope/etl/issues/5>.

Pour résumer :

Changement de forme Réorganisation du repo

J'ai essayé de faire du tri dans tous les codes, et ce que je propose comme orga du repo, c'est :

  • /data avec des échantillons de données d'entrées possibles (pour les tests)
  • /etl qui contient l'ETL principale et les utilitaires dans un paquet /etl/utils
  • docs: avec la docs (source & build)
  • /tests: qui contiendra les tests unitaires
  • /azure: qui contiendra les images docker & co ? (pas sûre de l'emplacement, @rcourivaudhttps://github.com/rcourivaud ?)
  • /wip_devel: avec les notebooks & co.

PEP 8 et doc-string

  • J'ai appliqué les changements nécessaires pour répondre aux standards Python
  • La doc-string est de type 'numpy' et nous permet de générer de la jolie doc avec Sphinx
  • J'ai essayé de compléter le README avec les infos qui manquaient

Pandas

  • J'utilise des DataFrame pandas, qui nous permettent d'avoir accès à pleins d'utilitaires cool. Par exemple, les données GPS ont maintenant cette tête là :

· latitude longitude elevation

·

· 2018-01-24 19:27:58+00:00 33.126515 -117.327168 -17.228

·

· 2018-01-24 19:27:59+00:00 33.126543 -117.327153 -18.199

·

· 2018-01-24 19:28:12+00:00 33.126616 -117.325690 -18.440

·

· 2018-01-24 19:28:13+00:00 33.126540 -117.327072 -19.161

·

· 2018-01-24 19:28:14+00:00 33.126539 -117.327218 -18.472

·

Misc

  • Print -> Logs
  • Camel case -> lower
  • ajout de requirements.txt et environment.yml
  • sphinx doc à build avec make html, qui ressemble à :

[Screenshot 2020-05-01 at 16 25 37]< https://user-images.githubusercontent.com/22340670/80812514-6b693b00-8bc8-11ea-9354-c643d7d80acb.png>

source build à compléter ! Optimisations

  • correction/optimisation de fillGPS La fonction fillGPS me semblait compliquée et sujet à erreurs (par ex, si le GPS et la vidéo commencent ou finissent pas ensemble). Puis, j'ai réalisé que les seuls points qui nous intéressaient étaient en fait ceux où il y a un déchet détecté. Donc, au lieu d'interpoler toutes les données manquantes du fichier GPS, je n'interpole que les données dont le timestamp correspond à un déchet détecté par l'IA (cf mon point correction de trashGPS ci-après).
  • changement d'ordre pour FAIL le plus tôt possible (sanity checks) Par ex, c'est dommage de faire un call à l'IA si à la fin la connection à la DB échoue. ou si on n'arrive pas à extraire les données GPS du fichier.
  • J'ai enlevé le script shell qui faisait appelle à une commande juste pour avoir la durée du média, et j'ai remplacer par une fonction python.
  • J'ai remplacé l'appel par ligne de commande à goproToGPx pour utiliser le paquet Pypi de Raph. D'ailleurs @rcourivaud< https://github.com/rcourivaud> , comment je fais pour fixer un truc sur le paquet ? (je trouve pas ton repo)
  • J'ai implémenter un CLI pour lancer l'ETL depuis le shell. C'est peut être un peu plus pratique que le parseargs (je trouve).

Nouvelles features Correction du trashGPS

  • On n'utilisait pas l'information 'frame_to_box' donnée par l'IA, qui donne le numero de la frame sur laquelle se trouve le trash. Etant donné que tu as aussi le fps et la durée, à partir du timestamp de début, on peut retrouver le timestamp du trash. A confirmer avec la team MOT (@charlesollionhttps://github.com/charlesollion ? ) si c'est bien ça, mais j'ai l'impression que oui. Du coup, ça permet d'optimiser la fonction qui prenait longtemps pour estimer les géométrie (X, Y, Z) ) partir de (latitude, longitude) car celle ci ne loop que sur les lignes où y a des déchets.

Accepte les entrées de type Smartphone vidéos et Manuelles

L'hypothèse faite ici, c'est qu'il y a 4 types de source d'acquisition possibles et que les fichiers GPX associés aux médias portent le même nom (@AntoineBrugehttps://github.com/AntoineBruge, ok?) :

  • gopro: fichier foo.mp4
  • smartphone video: fichier foo.mov + fichier foo.gpx (@AntoineBruge< https://github.com/AntoineBruge> , quelles autres extensions ?)
  • smartphone photo: fichier foo.jpg + fichier foo.gpx -manuelle: fichier foo.gpx

NB: La sortie de la version manuelle est légèrement différente:

  • colones pour les sorties gopro et smartphones : ['time', 'longitude', 'latitude', 'elevation', 'id', 'label', 'box', 'frame', 'geom']
  • colones pour les sorties manuelles : [time', 'longitude', 'latitude', 'label', 'geom']

Pouvoir travailler 100% en ligne

En fait, quand on veut tester ou développer, c'est pratique de ne pas être dépendant de Postgre et de Azure. Donc tu verras, j'ai essayé de documenter, mais dans la commande pour lancer l'ETL, on peut choisir de travailler avec des données locales plutôt que de télécharger depuis Azure (on pourra comme ça imaginer à l'avenir télécharger depuis d'autres sources). Pareil pour la sortie, on peut choisir de sauver le résultat en CSV localement plutôt que de l'insérer dans la base de données.

Todo, next steps

  • Voir comment télécharger un seul fichier du blob
  • Voir comment noter que l'ETL a déjà tourné sur un média, si ça a SUCCESS ou FAIL, ainsi que sa version, celle de l'IA.
  • Quelques fix dans le paquet de Raph qui extrait les données GPX
  • Ajout de tests unitaires
  • Porter sur Azure fonction

Qu'en dites vous ?

++

You can view, comment on, or merge this pull request online at:

https://github.com/surfriderfoundationeurope/etl/pull/6

Commit Summary

  • Code review, work in progress
  • Move functions in utils package
  • Major style refactoring
  • Add generic exception specific to ETL
  • Add .env to gitignore
  • Remove MediaInfo
  • Move requirements.txt and environment.yml to root
  • Handle file from smartphone, where GPX are given separately
  • Change path of gpx output
  • Rename modules
  • Assert conn is not None
  • Add CLI to launch th ETL flow
  • Re-organize repo and add version
  • Move samples in dedicated directory
  • Add media helpers
  • Black PEP.8 autom code format
  • Handle trash time interpolation and compute geom afterwards
  • simple quotes (black)
  • Add example of trashes answer from AI
  • Reorganize samples
  • Add sphinx doc
  • Fix docstring
  • Readme typo
  • Reorganization : move wip codes in directory 'devel'
  • Reorganization : move wip codes in directory 'wip_devel'
  • Move get_media_duration to media
  • Example .env file
  • Allow saving in local storage (for devel purpose)
  • Build Sphinx documentation
  • remove DS_Store
  • remove cached directory
  • Update README
  • Add docstring
  • Update doc, new build

File Changes

Patch Links:

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub< https://github.com/surfriderfoundationeurope/etl/pull/6>, or unsubscribe< https://github.com/notifications/unsubscribe-auth/ALB2U2Z2525GTAWNTWLWHTTRPLOKNANCNFSM4MXEQ45A>.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/surfriderfoundationeurope/etl/pull/6#issuecomment-623540564, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACDYPVPDVDA23GYVOSPLFK3RP3O4DANCNFSM4MXEQ45A .

-- Clément

cl3m3nt commented 4 years ago

Dear @bertrandlalo, thanks for the hard work. Following our call, please find some feebacks in italic related to your PR in front of each section:

Changement de forme

Réorganisation du repo

Your proposal makes sense and the reorganisation could be done later on J'ai essayé de faire du tri dans tous les codes, et ce que je propose comme orga du repo, c'est :

PEP 8 et doc-string

Ok to apply Numpy docstring I like the idea of having Sphinx documentation, this could be done later alongside repository reorganisation

Pandas

I find the idea of introducing Pandas dataframe very interesting to better fit to a data pipeline process. I want to evaluate pros/cons versus list/dico data structure but definitely worth to consider it

Misc

No pb to convert print to logs as well as Came case to lower Can you double check requirements.txt ? I had to add python-dotenv package to it to be able to successfully run the etl after building the Docker Image.

Optimisations

There are two important changes here that we could merge asap: shell script removal for getMediaInfo as well as goproToGPx package from Raph

Nouvelles features

Correction du trashGPS

Accepte les entrées de type Smartphone vidéos et Manuelles

L'hypothèse faite ici, c'est qu'il y a 4 types de source d'acquisition possibles et que les fichiers GPX associés aux médias portent le même nom (@AntoineBruge, ok?) :

NB1: La sortie de la version manuelle est légèrement différente:

NB2: Les labels des données manuelles sont différents : {'Objet vie courante', 'Bouteille boisson', 'Peche et chasse', 'Autre dechet', 'Industriel ou construction', 'Emballage alimentaire', 'Dechet agricole', 'Autres dechets +10'}

Pour tester ça en local, vous pouvez faire (après avoir activé l'environnement, cf README.rst) :

Pouvoir travailler 100% en local (sans se connecter à Azure & Postgre)

En fait, quand on veut tester ou développer, c'est pratique de ne pas être dépendant de Postgre et de Azure. Donc tu verras, j'ai essayé de documenter, mais dans la commande pour lancer l'ETL, on peut choisir de travailler avec des données locales plutôt que de télécharger depuis Azure (on pourra comme ça imaginer à l'avenir télécharger depuis d'autres sources). Pareil pour la sortie, on peut choisir de sauver le résultat en CSV localement plutôt que de l'insérer dans la base de données.

Todo, next steps

Qu'en dites vous ?

++

cl3m3nt commented 4 years ago

Dear @bertrandlalo, following previous feedbacks, my recommendation to integrated smoothly first updates is for you to open a new PR with:

This is an important step as this will allow us to remove dependancies to subprocess call as well as to gopro2gpx 3rd party. Tx

bertrandlalo commented 4 years ago

@cl3m3nt @AntoineBruge Juste un petit message, Clément, pour te dire que ce n'est pas très cool de :

C'est dommage, j'aurais bien aimé faire une team ETL... le code, c'est tellement mieux à plusieurs !

cl3m3nt commented 4 years ago

Bonjour Raphaelle,

je suis surpris du contenu de ton message, alors que je t'avais fait part de ma volonté d'intégrer des propositions progressivement. Si tu le souhaites, on peut en reparler afin d'évoquer la frustration que j'ai ressentie dans ton mail. Nous pensions justement avec Antoine te faire signe pour un nouveau besoin ponctuel. Qu'en penses tu ? Je peux trouver des disponibilités la semaine prochaine pour en parler.

Clement

On Tue, Jul 21, 2020 at 7:04 PM Raphaëlle Bertrand-Lalo < notifications@github.com> wrote:

@cl3m3nt https://github.com/cl3m3nt @AntoineBruge https://github.com/AntoineBruge Juste un petit message, Clément, pour te dire que ce n'est pas très cool de :

  • fermer une Pull-Request sans en informer l'auteur
  • prendre la quasi entièreté du code de la Pull-Request en question et le re-commit comme s'il était de toi
  • accepter ses propres Pull-Request... c'est presque absurde !

C'est dommage, j'aurais bien aimé faire une team ETL... le code, c'est tellement mieux à plusieurs !

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/surfriderfoundationeurope/etl/pull/6#issuecomment-661985688, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACDYPVLOVYCQ373LWULDAE3R4XDB7ANCNFSM4MXEQ45A .

-- Clément

bertrandlalo commented 4 years ago

Hello Clément,

Sur GitHub, tu peux accepter certains bouts (commit) d’une Pull-Request sans pour autant les re-commit. La différence se trouve dans la forme plus que dans le fond, c’est à dire que la contribution en question est laissée a l’auteur original.

C’est gentil de vouloir me re-integrer au projet après deux mois de silence, mais j’ai l’impression que le travail d’équipe est compliqué. J’étais super motivée mais je t’avoue que j’ai été déçue/frustrée par l’accueil de mon taffe.

Bonne suite dans l’ETL, j’espère quand même que tu trouveras quelqu’un pour reviewer le code, car je continue de penser que c’est risqué de vouloir être seul sur une brique, surtout dans un projet bénévole où l’équipe est censée être mouvante. Je vous laisse en discuter ensemble.

Bonne soirée !

Raphaëlle

Le mer. 22 juil. 2020 à 18:41, cl3m3nt notifications@github.com a écrit :

Bonjour Raphaelle,

je suis surpris du contenu de ton message, alors que je t'avais fait part de ma volonté d'intégrer des propositions progressivement. Si tu le souhaites, on peut en reparler afin d'évoquer la frustration que j'ai ressentie dans ton mail. Nous pensions justement avec Antoine te faire signe pour un nouveau besoin ponctuel. Qu'en penses tu ? Je peux trouver des disponibilités la semaine prochaine pour en parler.

Clement

On Tue, Jul 21, 2020 at 7:04 PM Raphaëlle Bertrand-Lalo < notifications@github.com> wrote:

@cl3m3nt https://github.com/cl3m3nt @AntoineBruge https://github.com/AntoineBruge Juste un petit message, Clément, pour te dire que ce n'est pas très cool de :

  • fermer une Pull-Request sans en informer l'auteur
  • prendre la quasi entièreté du code de la Pull-Request en question et le re-commit comme s'il était de toi
  • accepter ses propres Pull-Request... c'est presque absurde !

C'est dommage, j'aurais bien aimé faire une team ETL... le code, c'est tellement mieux à plusieurs !

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub < https://github.com/surfriderfoundationeurope/etl/pull/6#issuecomment-661985688 , or unsubscribe < https://github.com/notifications/unsubscribe-auth/ACDYPVLOVYCQ373LWULDAE3R4XDB7ANCNFSM4MXEQ45A

.

-- Clément

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/surfriderfoundationeurope/etl/pull/6#issuecomment-662560995, or unsubscribe https://github.com/notifications/unsubscribe-auth/AFKOIPWCKP22MG5SHDDMIX3R44JBZANCNFSM4MXEQ45A .

bertrandlalo commented 4 years ago

Le mer. 22 juil. 2020 à 19:12, Raphaelle Bertrand r.bertrand.lalo@gmail.com a écrit :

Hello Clément,

Sur GitHub, tu peux accepter certains bouts (commit) d’une Pull-Request sans pour autant les re-commit. La différence se trouve dans la forme plus que dans le fond, c’est à dire que la contribution en question est laissée a l’auteur original.

C’est gentil de vouloir me re-integrer au projet après deux mois de silence, mais j’ai l’impression que le travail d’équipe est compliqué. J’étais super motivée mais je t’avoue que j’ai été déçue/frustrée par l’accueil de mon taffe.

Bonne suite dans l’ETL, j’espère quand même que tu trouveras quelqu’un pour reviewer le code, car je continue de penser que c’est risqué de vouloir être seul sur une brique, surtout dans un projet bénévole où l’équipe est censée être mouvante. Je vous laisse en discuter ensemble.

Bonne soirée !

Raphaëlle

Le mer. 22 juil. 2020 à 18:41, cl3m3nt notifications@github.com a écrit :

Bonjour Raphaelle,

je suis surpris du contenu de ton message, alors que je t'avais fait part de ma volonté d'intégrer des propositions progressivement. Si tu le souhaites, on peut en reparler afin d'évoquer la frustration que j'ai ressentie dans ton mail. Nous pensions justement avec Antoine te faire signe pour un nouveau besoin ponctuel. Qu'en penses tu ? Je peux trouver des disponibilités la semaine prochaine pour en parler.

Clement

On Tue, Jul 21, 2020 at 7:04 PM Raphaëlle Bertrand-Lalo < notifications@github.com> wrote:

@cl3m3nt https://github.com/cl3m3nt @AntoineBruge https://github.com/AntoineBruge Juste un petit message, Clément, pour te dire que ce n'est pas très cool de :

  • fermer une Pull-Request sans en informer l'auteur
  • prendre la quasi entièreté du code de la Pull-Request en question et le re-commit comme s'il était de toi
  • accepter ses propres Pull-Request... c'est presque absurde !

C'est dommage, j'aurais bien aimé faire une team ETL... le code, c'est tellement mieux à plusieurs !

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub < https://github.com/surfriderfoundationeurope/etl/pull/6#issuecomment-661985688 , or unsubscribe < https://github.com/notifications/unsubscribe-auth/ACDYPVLOVYCQ373LWULDAE3R4XDB7ANCNFSM4MXEQ45A

.

-- Clément

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/surfriderfoundationeurope/etl/pull/6#issuecomment-662560995, or unsubscribe https://github.com/notifications/unsubscribe-auth/AFKOIPWCKP22MG5SHDDMIX3R44JBZANCNFSM4MXEQ45A .

cl3m3nt commented 4 years ago

Bonjour Raphaelle,

ok je comprends mieux. Si tu changes d'avis, n'hésite pas à me faire signe. Il y a deux sujets intéressants que je pensais pouvoir te proposer.

Clement

On Wed, Jul 22, 2020 at 7:13 PM Raphaëlle Bertrand-Lalo < notifications@github.com> wrote:

Le mer. 22 juil. 2020 à 19:12, Raphaelle Bertrand < r.bertrand.lalo@gmail.com> a écrit :

Hello Clément,

Sur GitHub, tu peux accepter certains bouts (commit) d’une Pull-Request sans pour autant les re-commit. La différence se trouve dans la forme plus que dans le fond, c’est à dire que la contribution en question est laissée a l’auteur original.

C’est gentil de vouloir me re-integrer au projet après deux mois de silence, mais j’ai l’impression que le travail d’équipe est compliqué. J’étais super motivée mais je t’avoue que j’ai été déçue/frustrée par l’accueil de mon taffe.

Bonne suite dans l’ETL, j’espère quand même que tu trouveras quelqu’un pour reviewer le code, car je continue de penser que c’est risqué de vouloir être seul sur une brique, surtout dans un projet bénévole où l’équipe est censée être mouvante. Je vous laisse en discuter ensemble.

Bonne soirée !

Raphaëlle

Le mer. 22 juil. 2020 à 18:41, cl3m3nt notifications@github.com a écrit :

Bonjour Raphaelle,

je suis surpris du contenu de ton message, alors que je t'avais fait part de ma volonté d'intégrer des propositions progressivement. Si tu le souhaites, on peut en reparler afin d'évoquer la frustration que j'ai ressentie dans ton mail. Nous pensions justement avec Antoine te faire signe pour un nouveau besoin ponctuel. Qu'en penses tu ? Je peux trouver des disponibilités la semaine prochaine pour en parler.

Clement

On Tue, Jul 21, 2020 at 7:04 PM Raphaëlle Bertrand-Lalo < notifications@github.com> wrote:

@cl3m3nt https://github.com/cl3m3nt @AntoineBruge https://github.com/AntoineBruge Juste un petit message, Clément, pour te dire que ce n'est pas très cool de :

  • fermer une Pull-Request sans en informer l'auteur
  • prendre la quasi entièreté du code de la Pull-Request en question et le re-commit comme s'il était de toi
  • accepter ses propres Pull-Request... c'est presque absurde !

C'est dommage, j'aurais bien aimé faire une team ETL... le code, c'est tellement mieux à plusieurs !

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <

https://github.com/surfriderfoundationeurope/etl/pull/6#issuecomment-661985688

, or unsubscribe <

https://github.com/notifications/unsubscribe-auth/ACDYPVLOVYCQ373LWULDAE3R4XDB7ANCNFSM4MXEQ45A

.

-- Clément

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub < https://github.com/surfriderfoundationeurope/etl/pull/6#issuecomment-662560995 , or unsubscribe < https://github.com/notifications/unsubscribe-auth/AFKOIPWCKP22MG5SHDDMIX3R44JBZANCNFSM4MXEQ45A

.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/surfriderfoundationeurope/etl/pull/6#issuecomment-662577078, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACDYPVPULEN75QUED4NXU43R44M37ANCNFSM4MXEQ45A .

-- Clément