Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Relectures d'une partie du chapitre 2, de l'INDEX et de README #19

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

pasco-py
Copy link

@pasco-py pasco-py commented May 2, 2024

Relectures d'une partie du chapitre 2, de l'INDEX et de README :

  • corrections mineures de mise en page (',', '., 'espace avant :'),
  • remplacement de 'SRC' par 'CRS',
  • quelques fautes d'orthographe,
  • quelques améliorations de traduction.

Par contre, désolé, je n'ai pas beaucoup d'expérience sur l'utilisation de GitHub / RStudio. Pendant la relecture du chapitre 2 sous RStudio, lors d'un passage d'un mode 'Source' à 'Visual', il m'a ajouté les lignes suivantes en début de code :

editor_options:
markdown:
wrap: sentence

cela a décalé les lignes de l'ensemble du code, et il me semble à ajouter de changements où il n'y en a pas... je m'en suis rendu compte après avoir pusher mes modifications.

Dites-moi si c'est ok ou s'il faut que je recommence à partir de la version précédente.

…ise en page, CRS (SRC), plus quelques fautes d'orthographes.
Copy link
Collaborator

@defuneste defuneste left a comment

Choose a reason for hiding this comment

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

Merci pour tout ces corrections! J'avoue que je suis sur un qwerty et que souvent je dois utiliser la "technique Beyoncé" pour certains accents ..

je retire les lignes, la raison principale est que @Nowosad et @Robinlovelace font assez attention a bien avoir une phrase / une ligne. Cela facilite le travail de suivit, si ils changent un truc, rajoute/supprime une ligne avec git et markdown c'est plus facile a reperer.
Ces colonnes spatiales sont souvent appelées `geom` ou `geometry` : ainsi `world$geom` fait référence à l'élément spatial de l'objet `world` décrit plus haut.
Ces colonnes géométriques sont des "colonnes de type liste" de la classe `sfc` (voir la section \@ref(sfc)).
Puis, à leur tour, les objets `sfc` sont composés d'un ou plusieurs objets de la classe `sfg` : des géométries d'entités simples que nous décriverons dans la section \@ref(sfg).
\index{sf!sfc}
\index{simple feature columns|see {sf!sfc}}
\index{sf!sfc} \index{simple feature columns|see {sf!sfc}}
Copy link
Collaborator

@defuneste defuneste May 3, 2024

Choose a reason for hiding this comment

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

Il faudra "resauter" les lignes ici pour suivre la numerotation de l'ouvrage en anglais. C'est une ruse du format markdown: le saut de ligne ne passe que sur un double white space ou un double retour de chariot et du coup sauter une ligne permet de mieux les suivre via git sans alterer le rendu du texte.

ie:

blabla
blibip

sera rendu blabla blibip

mais git traque deux lignes, ainsi juste modifier "blabla" ne touche pas "blibip"

(je sais pas si je suis clair ...)

Copy link
Author

Choose a reason for hiding this comment

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

OK. Oui, j'ai compris !

@defuneste
Copy link
Collaborator

The error in the GHA come from sourcing the chunk [raster-intro-plot] from the english repo (https://github.com/geocompx/geocompr/blob/main/code/02-raster-intro-plot.R).

I will need to review that tomorrow. Any idea @Robinlovelace ?

Stuff that could be incorrect:

  • I am never sure how function namespace are loaded inside a chunk that is sourcing some code (yeah I need to read the doc from source() all the time...).

  • The code is loading tmap, so it could be that tm_tilte is a new introduction in {tmap} and the version that the french repo is running is too old.

  • other troubles that I did not think off.

Anyway I am excited that @pasco-py took time to review and correct a bunch of stuff!

@Robinlovelace
Copy link
Contributor

I will need to review that tomorrow. Any idea @Robinlovelace ?

Will take a look..

@Robinlovelace
Copy link
Contributor

Cause: changes in {tmap} I think.

Relevant bit of code in the English version:

 raster-intro-plot, echo = FALSE, fig.cap = "Raster data types: (A) cell IDs, (B) cell values, (C) a colored raster map.", fig.scap="Raster data types.", fig.asp=0.5, message=FALSE}
source("https://github.com/geocompx/geocompr/raw/main/code/02-raster-intro-plot.R", print.eval = TRUE)

In your version, TBC, here's the message that led me there: https://github.com/geocompx/geocompr-fr/actions/runs/8932868877/job/24537379475?pr=19#step:4:238

@pasco-py
Copy link
Author

pasco-py commented May 3, 2024

Merci pour tout ces corrections! J'avoue que je suis sur un qwerty et que souvent je dois utiliser la "technique Beyoncé" pour certains accents ..

OK ! Je vais prendre le temps de relire l'ensemble des chapitres 2 à 7. Je ferai des « pull request » au fur et à mesure.

@defuneste
Copy link
Collaborator

Hey @Robinlovelace yup that my guess I will try to sneak some time today to correct that! thanks!

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.

3 participants