Closed stecaron closed 3 years ago
Il y a probablement des fautes de français, mais je pense que c'est prêt pour une premiere relecture par quelqu'un 😊 @jplecavalier @samperochkin . Ça dit "25 minutes de lecture", mais je suis un peu impacter par mes chunks de code et mes figures ... ça se lit assez vite j'ai l'impression.
Excellent article, possibilité de reach large.
Par contre, desfois il y a beaucoup de "jargon" hockey/analytique qui serait pertinent de définir et l'article aurait un + gros potentiel de reach. Mais définitivement un gros article.
Ouais je comprends ton point @davebulaval ! Je pense que considérant qu'il y a deja beaucoup de contenu, je préfère presque réduire un peu le reach, mais éviter d'avoir un article trop long a lire (et ainsi perdre le monde que je voulais reach a la base). Bon à savoir pour un prochain article par contre. Peut-être penser a des "Part 1" et "Part 2" la prochaine fois ..
Excellent article, possibilité de reach large. Par contre, desfois il y a beaucoup de "jargon" hockey/analytique qui serait pertinent de définir et l'article aurait un + gros potentiel de reach. Mais définitivement un gros article.
Ouais je comprends ton point @davebulaval ! Je pense que considérant qu'il y a deja beaucoup de contenu, je préfère presque réduire un peu le reach, mais éviter d'avoir un article trop long a lire (et ainsi perdre le monde que je voulais reach a la base). Bon à savoir pour un prochain article par contre. Peut-être penser a des "Part 1" et "Part 2" la prochaine fois ..
Oui, ou sinon pointer vers le terme maybe?
On considère ça good to go?
On considère ça good to go?
Je vais faire une lecture finale maybe et je pense c'est good to go!
Je pense que @jplecavalier voulait lire aussi ce soir. On peut attendre avant de merge voir s'il a des commentaires, surtout que c'est son package donc si j'ai chié de quoi dans l'extraction/manipulations des données il devrait le voir ...
Quand même déçu que @davebulaval t'ailles pas fait de commentaires positifs sur mon gif de LOTR ...
Quand même déçu que @davebulaval t'ailles pas fait de commentaires positifs sur mon gif de LOTR ...
J'ai pas vu tout assembler comme c'est un fork de ton repo, juste vu le raw text.
Je relis ça attentivement ce soir sans faute. J'vais moins être focus sur le texte et me concentrer sur le code comme vous avez déjà pas mal review le texte.
Quand même déçu que @davebulaval t'ailles pas fait de commentaires positifs sur mon gif de LOTR ...
J'ai pas vu tout assembler comme c'est un fork de ton repo, juste vu le raw text.
J'pensais t'avais checker le preview aussi ...
Quand même déçu que @davebulaval t'ailles pas fait de commentaires positifs sur mon gif de LOTR ...
J'ai pas vu tout assembler comme c'est un fork de ton repo, juste vu le raw text.
J'pensais t'avais checker le preview aussi ...
Ben crisse, tu viens de m'apprendre ça. Je seconde la référence LOTR
Maintenant qu'on a appris à Dave l'existence des previews Netlify, il va falloir que vous m'appreniez comment faire des suggestions de modification qui peuvent directement être acceptées, je suis un noob en review faut croire, j'ai tout écrit en simple commentaires. Thanks @davebulaval d'avoir edit mes comments.
Je dis ça de même, si jamais vous voulez être capable de configurer manuellement le reading time pour les articles contenant beaucoup de code HTML in-line (ce qui fait exploser l'algo de calcul), vous devrez modifier les partials du site. Vous pouvez vous inspirer de ce que j'ai fait ici (voir les lignes 34-38)
Maintenant qu'on a appris à Dave l'existence des previews Netlify, il va falloir que vous m'appreniez comment faire des suggestions de modification qui peuvent directement être acceptées, je suis un noob en review faut croire, j'ai tout écrit en simple commentaires. Thanks @davebulaval d'avoir edit mes comments.
Quand tu edit sur la plateforme t'a cet icone là
qui permet de faire des quick fix en commit. Tu peux aussi drag un bout de code (si c'Est plusieurs ligne) pour edit plusieurs section.
Je dis ça de même, si jamais vous voulez être capable de configurer manuellement le reading time pour les articles contenant beaucoup de code HTML in-line (ce qui fait exploser l'alto de calcul), vous devrez modifier les partials du site. Vous pouvez vous inspirer de ce que j'ai fait ici (voir les lignes 34-38)
Bon point, est-ce que ça veut dire que si tu spécifies rien dans ton header ça va faire le calcul comme en ce moment?
Je dis ça de même, si jamais vous voulez être capable de configurer manuellement le reading time pour les articles contenant beaucoup de code HTML in-line (ce qui fait exploser l'alto de calcul), vous devrez modifier les partials du site. Vous pouvez vous inspirer de ce que j'ai fait ici (voir les lignes 34-38)
Bon point, est-ce que ça veut dire que si tu spécifies rien dans ton header ça va faire le calcul comme en ce moment?
Oui, exactement. Comment je l'ai configuré sur mon site, quand j'indique un paramètre reading_time
dans mon YAML, c'est ça qui est utilisé, sinon ça fall back au comportement par défaut de Hugo.
Rendu où dans la correction? Ça me semble final pour un release mercredi prochain ça!
Je viens de push les derniers commentaires ! Resterait peut-être a revoir les partials pour ajuster le reading_time .. Je m'y connais pas trop là-dedans, j'ai checker le fichier que JP a mis en référence, on dirait qu'on a pas ce fichier là dans nos partials, je veux pas ramener des trucs qu'on veut pas ramener haha ...
Je dis ça de même, si jamais vous voulez être capable de configurer manuellement le reading time pour les articles contenant beaucoup de code HTML in-line (ce qui fait exploser l'alto de calcul), vous devrez modifier les partials du site. Vous pouvez vous inspirer de ce que j'ai fait ici (voir les lignes 34-38)
Bon point, est-ce que ça veut dire que si tu spécifies rien dans ton header ça va faire le calcul comme en ce moment?
Oui, exactement. Comment je l'ai configuré sur mon site, quand j'indique un paramètre
reading_time
dans mon YAML, c'est ça qui est utilisé, sinon ça fall back au comportement par défaut de Hugo.
@AnnieDeshaies any clues? (https://github.com/jplecavalier/personal-website/blob/dc766df882462283056b14cf90e6b00642fa7891/layouts/partials/page_metadata.html#L34)
Je viens de push les derniers commentaires ! Resterait peut-être a revoir les partials pour ajuster le reading_time .. Je m'y connais pas trop là-dedans, j'ai checker le fichier que JP a mis en référence, on dirait qu'on a pas ce fichier là dans nos partials, je veux pas ramener des trucs qu'on veut pas ramener haha ...
@stecaron Je viens de merge le fix avec @AnnieDeshaies.
@davebulaval, @AnnieDeshaies vous êtes des machines. J'ai voici donc avec mon Reading time mis à jour.
One last thing ULTRA picky mais pour less graphes je prendrais les couleurs .Layer comme @jeremiedb as fait dans ces arbres: https://deploy-preview-138--relaxed-kepler-4b9238.netlify.app/blog/2020-01-04-julia-boosting-trees/julia-boosting-trees/
As far as I'm concerned, je pense c'est ready to be MRG. @davebulaval j'ai pas ajouté les couleurs dans les graphs, j'trouve ca fait un peu agressant pour des graphiques. My 2 cents ...
Good for me also!
Good for me also!
@stecaron post demain confirmer. Peroch post sur FB moi je fais le post LinkedIn.
Encore quelques petits trucs mineurs à finaliser, mais je crée le PR quand même. Je changerai le tag [WIP] to [MRG] quand ce sera final et prêt à être relu.