Relecture #2

guillaumefe · 2019-08-22T02:58:28Z

Relecture du document python_profiling.ipynb

R1 : Correction de quelques coquilles
Traduction française partout où c'était possible
R2 : Mise au format (voir Reference File)
R3 : Cosmétique, tournures

branche de travaill : contribution-guillaume

virtualenv relecture
python3.5
jupyter-notebook : 6.0.0
Linux version 4.9.0-6-686-pae

MordicusEtCubitus

Merci pour les corrections de fautes que je valide, mais ton code change aussi les numéros des cellules ou les messages de sortie, ce que je ne souhaite pas. Peux-tu le revoir ?

MordicusEtCubitus · 2019-08-26T09:03:40Z

profiling/python_profiling.ipynb

   "metadata": {},
   "outputs": [
    {
     "data": {
      "text/plain": [
-       "24.584196265001083"


Je ne souhaite pas modifier mes sorties

MordicusEtCubitus · 2019-08-26T09:04:09Z

profiling/python_profiling.ipynb

@@ -167,16 +167,16 @@
  },
  {
   "cell_type": "code",
-   "execution_count": 4,


Je ne souhaite pas modifier les numéros de cellules exécutées

MordicusEtCubitus · 2019-08-26T09:04:32Z

profiling/python_profiling.ipynb

-     ]
-    }
-   ],
+   "outputs": [],


Je ne souhaite pas supprimer les sorties

MordicusEtCubitus · 2019-08-26T09:05:55Z

profiling/python_profiling.ipynb

+     "evalue": "name 'intersection' is not defined",
+     "output_type": "error",
+     "traceback": [
+      "\u001b[0;31m---------------------------------------------------------------------------\u001b[0m",


Apparemment tu as eu cette erreur parce que tu as repris le notebook sans ré-exécuter les premières cellules
Je ne peux pas intégrer cette erreur

MordicusEtCubitus

Peux-tu corriger les éléments commentés afin que ce commit n'inclue vraiment que les fautes d'orthographe ?

MordicusEtCubitus · 2019-08-26T09:08:23Z

profiling/python_profiling.ipynb

    "\n",
    "Le résultat peut être visualisé sous forme de tableau."
   ]
  },
  {
   "cell_type": "code",
-   "execution_count": 21,
+   "execution_count": 5,


Je ne veux pas changer les numéros de cellules

MordicusEtCubitus · 2019-08-26T09:08:46Z

profiling/python_profiling.ipynb

   "metadata": {},
   "outputs": [
    {
     "name": "stdout",
     "output_type": "stream",
     "text": [
-      "         83650 function calls (4 primitive calls) in 0.011 seconds\n",
+      "         3 function calls in 0.000 seconds\n",


Je ne souhaite pas modifier les sorties

MordicusEtCubitus · 2019-08-26T09:09:19Z

profiling/python_profiling.ipynb

+      "\u001b[0;31mNameError\u001b[0m                                 Traceback (most recent call last)",
+      "\u001b[0;32m<ipython-input-5-04b57558cc95>\u001b[0m in \u001b[0;36m<module>\u001b[0;34m\u001b[0m\n\u001b[1;32m      1\u001b[0m \u001b[0;32mimport\u001b[0m \u001b[0mcProfile\u001b[0m\u001b[0;34m\u001b[0m\u001b[0m\n\u001b[0;32m----> 2\u001b[0;31m \u001b[0mcProfile\u001b[0m\u001b[0;34m.\u001b[0m\u001b[0mrun\u001b[0m\u001b[0;34m(\u001b[0m\u001b[0;34m\"padovan(39)\"\u001b[0m\u001b[0;34m)\u001b[0m\u001b[0;34m\u001b[0m\u001b[0m\n\u001b[0m",
+      "\u001b[0;32m/usr/lib/python3.5/cProfile.py\u001b[0m in \u001b[0;36mrun\u001b[0;34m(statement, filename, sort)\u001b[0m\n\u001b[1;32m     14\u001b[0m \u001b[0;34m\u001b[0m\u001b[0m\n\u001b[1;32m     15\u001b[0m \u001b[0;32mdef\u001b[0m \u001b[0mrun\u001b[0m\u001b[0;34m(\u001b[0m\u001b[0mstatement\u001b[0m\u001b[0;34m,\u001b[0m \u001b[0mfilename\u001b[0m\u001b[0;34m=\u001b[0m\u001b[0;32mNone\u001b[0m\u001b[0;34m,\u001b[0m \u001b[0msort\u001b[0m\u001b[0;34m=\u001b[0m\u001b[0;34m-\u001b[0m\u001b[0;36m1\u001b[0m\u001b[0;34m)\u001b[0m\u001b[0;34m:\u001b[0m\u001b[0;34m\u001b[0m\u001b[0m\n\u001b[0;32m---> 16\u001b[0;31m     \u001b[0;32mreturn\u001b[0m \u001b[0m_pyprofile\u001b[0m\u001b[0;34m.\u001b[0m\u001b[0m_Utils\u001b[0m\u001b[0;34m(\u001b[0m\u001b[0mProfile\u001b[0m\u001b[0;34m)\u001b[0m\u001b[0;34m.\u001b[0m\u001b[0mrun\u001b[0m\u001b[0;34m(\u001b[0m\u001b[0mstatement\u001b[0m\u001b[0;34m,\u001b[0m \u001b[0mfilename\u001b[0m\u001b[0;34m,\u001b[0m \u001b[0msort\u001b[0m\u001b[0;34m)\u001b[0m\u001b[0;34m\u001b[0m\u001b[0m\n\u001b[0m\u001b[1;32m     17\u001b[0m \u001b[0;34m\u001b[0m\u001b[0m\n\u001b[1;32m     18\u001b[0m \u001b[0;32mdef\u001b[0m \u001b[0mrunctx\u001b[0m\u001b[0;34m(\u001b[0m\u001b[0mstatement\u001b[0m\u001b[0;34m,\u001b[0m \u001b[0mglobals\u001b[0m\u001b[0;34m,\u001b[0m \u001b[0mlocals\u001b[0m\u001b[0;34m,\u001b[0m \u001b[0mfilename\u001b[0m\u001b[0;34m=\u001b[0m\u001b[0;32mNone\u001b[0m\u001b[0;34m,\u001b[0m \u001b[0msort\u001b[0m\u001b[0;34m=\u001b[0m\u001b[0;34m-\u001b[0m\u001b[0;36m1\u001b[0m\u001b[0;34m)\u001b[0m\u001b[0;34m:\u001b[0m\u001b[0;34m\u001b[0m\u001b[0m\n",
+      "\u001b[0;32m/usr/lib/python3.5/profile.py\u001b[0m in \u001b[0;36mrun\u001b[0;34m(self, statement, filename, sort)\u001b[0m\n\u001b[1;32m     53\u001b[0m         \u001b[0mprof\u001b[0m \u001b[0;34m=\u001b[0m \u001b[0mself\u001b[0m\u001b[0;34m.\u001b[0m\u001b[0mprofiler\u001b[0m\u001b[0;34m(\u001b[0m\u001b[0;34m)\u001b[0m\u001b[0;34m\u001b[0m\u001b[0m\n\u001b[1;32m     54\u001b[0m         \u001b[0;32mtry\u001b[0m\u001b[0;34m:\u001b[0m\u001b[0;34m\u001b[0m\u001b[0m\n\u001b[0;32m---> 55\u001b[0;31m             \u001b[0mprof\u001b[0m\u001b[0;34m.\u001b[0m\u001b[0mrun\u001b[0m\u001b[0;34m(\u001b[0m\u001b[0mstatement\u001b[0m\u001b[0;34m)\u001b[0m\u001b[0;34m\u001b[0m\u001b[0m\n\u001b[0m\u001b[1;32m     56\u001b[0m         \u001b[0;32mexcept\u001b[0m \u001b[0mSystemExit\u001b[0m\u001b[0;34m:\u001b[0m\u001b[0;34m\u001b[0m\u001b[0m\n\u001b[1;32m     57\u001b[0m             \u001b[0;32mpass\u001b[0m\u001b[0;34m\u001b[0m\u001b[0m\n",


Idem, ici tu n'as pas exécuté les premières cellules, du coup le code ne passe pas

MordicusEtCubitus · 2019-08-26T09:14:14Z

profiling/python_profiling.ipynb

@@ -55,7 +55,7 @@
    "\n",
    "On dénote en général 2 types de profileurs:\n",
    "\n",
-    "* Les profileurs événementiels ou déterministes (tracing profiler), qui enregistrent toutes les actions qui se passent dans le programme et peuvent fournir beaucoup de statistiques très précises.   \n",
+    "* Les profileurs événementiels ou déterministes (*Tracing Profiler*), qui enregistrent toutes les actions qui se passent dans le programme et peuvent fournir beaucoup de statistiques très précises.   \n",


Ici, c'est ok
Sais-tu si je peux approver juste ce commit ? Je vais regarder dans la doc...

MordicusEtCubitus · 2019-08-26T09:19:24Z

profiling/python_profiling.ipynb

@@ -80,7 +80,7 @@
  },
  {
   "cell_type": "code",
-   "execution_count": 1,
+   "execution_count": 7,


Je ne souhaite pas changer les numéros de cellules

MordicusEtCubitus · 2019-08-26T09:19:47Z

profiling/python_profiling.ipynb

   "metadata": {},
   "outputs": [
    {
     "data": {
      "text/plain": [
-       "66.20003481100139"
+       "68.96059255599903"


Je ne souhaite pas changer les sorties

MordicusEtCubitus · 2019-08-26T09:21:30Z

profiling/python_profiling.ipynb

-     "output_type": "execute_result"
-    }
-   ],
+   "outputs": [],


Pourquoi veux-tu supprimer le graphique ?

MordicusEtCubitus · 2019-08-26T09:22:42Z

profiling/python_profiling.ipynb

   "source": [
    "from graphviz import Digraph\n",
    "\"\"\"\n",
-    "To install graphviz\n",


En fait je pensais laisser les commentaires en anglais, ainsi les non francophones peuvent quand-même comprendre le code à défaut des explications textuelles.

MordicusEtCubitus · 2019-08-26T09:31:17Z

profiling/python_profiling.ipynb

@@ -18,7 +18,7 @@
   "cell_type": "markdown",
   "metadata": {},
   "source": [
-    "> Les boutons *Solution* empêchent le bon rendu de ce notebook par **github** ou *nbviewer*, vous pouvez utiliser ce lien **[MyBinder](https://mybinder.org/v2/gh/MordicusEtCubitus/CoursPython/master?filepath=profiling%2Fpython_profiling.ipynb)** pour une bonne visualisation."
+    "> Les boutons *Solution* empêchent le bon rendu de ce notebook par **github** ou *nbviewer*, vous êtes invités à utiliser ce lien **[MyBinder](https://mybinder.org/v2/gh/MordicusEtCubitus/CoursPython/master?filepath=profiling%2Fpython_profiling.ipynb)** pour bénéficier d'une meilleure visualisation."


Ok, pour changer la formulation, mais dans ce cas je préfère

Il est conseillé d'utiliser ce lien, car cela ne tient pas compte du genre masculin/féminin.
Dans ta formulation il faudrait écrire vous êtes invité(e)

MordicusEtCubitus · 2019-08-26T09:32:05Z

profiling/python_profiling.ipynb

@@ -51,15 +51,13 @@
   "cell_type": "markdown",
   "metadata": {},
   "source": [
-    "## Types de profiler\n",


même Pofileurs

MordicusEtCubitus · 2019-08-26T09:36:32Z

profiling/python_profiling.ipynb

   "metadata": {},
-   "outputs": [
-    {
-     "name": "stdout",


Toujours les mêmes remarques sur les numéros de cellules et sorties

MordicusEtCubitus · 2019-08-26T09:37:37Z

profiling/python_profiling.ipynb

@@ -78,7 +78,7 @@
  },
  {
   "cell_type": "code",
-   "execution_count": 20,


Je ne comprends pas ce commit

MordicusEtCubitus · 2019-08-26T09:40:47Z

Merci beaucoup pour tes efforts et ta relecture
Si tu peux restituer les numéros de cellules, sorties et graphiques j'intègre le tout

guillaumefe added 8 commits August 21, 2019 18:46

R1 - coquilles

84ade75

R2 - Format

8570512

R2 - Format Reference File

92a7946

frenched comments

89c32df

forgotten In English Word format

e8606d8

R3 - cosmetique

9d14073

MAJ reference file

83d559c

cheked all run ok

e6a9846

MordicusEtCubitus requested changes Aug 22, 2019

View reviewed changes

MordicusEtCubitus reviewed Aug 26, 2019

View reviewed changes

profiling/python_profiling.ipynb Outdated

]

}

],

"outputs": [],

Copy link

Owner

MordicusEtCubitus Aug 26, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Je ne souhaite pas supprimer les sorties

MordicusEtCubitus reviewed Aug 26, 2019

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Relecture #2

Relecture #2

guillaumefe commented Aug 22, 2019

MordicusEtCubitus left a comment

MordicusEtCubitus Aug 26, 2019

MordicusEtCubitus Aug 26, 2019

MordicusEtCubitus Aug 26, 2019

MordicusEtCubitus Aug 26, 2019

MordicusEtCubitus left a comment

MordicusEtCubitus Aug 26, 2019

MordicusEtCubitus Aug 26, 2019

MordicusEtCubitus Aug 26, 2019

MordicusEtCubitus Aug 26, 2019

MordicusEtCubitus Aug 26, 2019

MordicusEtCubitus Aug 26, 2019

MordicusEtCubitus Aug 26, 2019

MordicusEtCubitus Aug 26, 2019

MordicusEtCubitus Aug 26, 2019 •

edited

Loading

MordicusEtCubitus Aug 26, 2019

MordicusEtCubitus Aug 26, 2019

MordicusEtCubitus Aug 26, 2019

MordicusEtCubitus commented Aug 26, 2019

Relecture #2

Are you sure you want to change the base?

Relecture #2

Conversation

guillaumefe commented Aug 22, 2019

Relecture du document python_profiling.ipynb

branche de travaill : contribution-guillaume

MordicusEtCubitus left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

MordicusEtCubitus left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

MordicusEtCubitus Aug 26, 2019 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

MordicusEtCubitus commented Aug 26, 2019

MordicusEtCubitus Aug 26, 2019 •

edited

Loading