Skip to content

Révision du chapitre 2 selon le feedback H26#10

Merged
plstonge merged 5 commits into
mainfrom
rev/h26/chap2
May 29, 2026
Merged

Révision du chapitre 2 selon le feedback H26#10
plstonge merged 5 commits into
mainfrom
rev/h26/chap2

Conversation

@plstonge
Copy link
Copy Markdown
Collaborator

  • Dans le dernier exercice, grouper selon species_id au lieu de plot_id
  • Simplifier le dernier exercice en pratiquant l'utilisation de la sélection plutôt que de pratiquer groupby().
  • Enlever la sélection selon [0, 10] et [0, 10, 35549]. Au pire, on aura des questions. Les exemples restants vont avec le premier exercice qui suit.
  • Enlever .shape après la sélection par isin() pour voir le résultat en entier.

@plstonge plstonge requested a review from dlq May 22, 2026 20:40
@dlq
Copy link
Copy Markdown
Contributor

dlq commented May 28, 2026

Je ne comprends pas le [0,10] -> 0:4. Pourquoi montrer 4 lignes en continu et non 2, non-continu?

Le reste est bien.

@plstonge
Copy link
Copy Markdown
Collaborator Author

C'était pour réduire la quantité de matériel et montrer ce qui serait utilisé dans l'exercice suivant.

En fait, le use-case où on sélectionne plusieurs enregistrements par leur numéro, je ne le connais pas vraiment ; ces numéros n'ont pas de signification. Peut-être qu'avec un autre index ou un autre jeu de données, ça aurait été plus pertinent.

Dis-moi ce que tu en penses : l'exercice fait une sélection de lignes avec un intervalle de 'NL':'PB'. Serait-il mieux de sélectionner des espèces précises au lieu d'un intervalle?

@dlq
Copy link
Copy Markdown
Contributor

dlq commented May 28, 2026

Hmm. Je ne sais pas!

@plstonge
Copy link
Copy Markdown
Collaborator Author

Voilà, j'ai ramené l'exemple [0, 10] et j'ai modifié l'exercice.

@plstonge plstonge merged commit 5016db0 into main May 29, 2026
@plstonge plstonge deleted the rev/h26/chap2 branch May 29, 2026 15:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants