Passage à zmarkdown 11.3.0 #6410
Conversation
Arnaud-D
added
the
hacktoberfest-accepted
|
@philippemilink tu peux le déployer sur la béta histoire qu'on ait des données réelles stp? |
|
C'est déployé ! |
|
Pour les dates ça semble bien réglé. a priori les math sont correctement gérées maintenant mais il y a une autre erreur, j'irai regarder les logs quand j'aurai un accès. |
|
Voici les liens vers les fichiers sources Markdown, LaTeX et les logs LaTeX :
En fait, l'affichage des tableaux complexes est cassé ! Un tableau aussi simple que celui-ci n'est pas rendu correctement par TeXLive : +-------+-----------------+
| Table | Headings Here |
+=======+=================+
| cell | column spanning |
+-------+-----------------+
| multi | cells can be |
+-------+-----------------+
|
|
Salut ici, le problème des tableaux est connu et non-lié à la version 11.1, il avait été soulevé par philippemilink ici : zestedesavoir/zmarkdown#463. Une résolution est prévue mais les tableaux... c'est compliqué. On avait commencé à regarder avec l'équipe du template pour voir comment gérer ça. En tout cas ça ne pose pas de bug de compilation théoriquement (si c'est le cas on est mal), donc il doit y avoir un soucis ailleurs. Peut-être que vous pourriez essayer de générer les exports du dernier billet/article de pierre-24 pour savoir si les maths refonctionnent, il était aussi concerné par le bug. À regarder rapidement les logs, je crois qu'on a un soucis avec l'échappement : un |
|
La génération du dernier billet de pierre_24 (celui sur le Wifi, c'est bien celui-là dont tu parlais ?) échoue aussi sur la bêta.
Cela semble être dû aussi au soucis d'échappement. |
|
Oui, c'est bien celui-ci dont je parlais, merci d'avoir testé, ça semble confirmer ce que je croyais. |
Situphen
added
the
En attente de la dépendance
|
Salut, une nouvelle version ZMarkdown 11.2.0, publiée à l'instant résoud théoriquement les problèmes d'échappements. La note de version complète est disponible ici : https://github.com/zestedesavoir/zmarkdown/releases/tag/zmarkdown%4011.2.0 |
Situphen
removed
the
En attente de la dépendance
|
Les requêtes vers zmarkdown échouent avec cette erreur : Tu as ajouté |
|
Sorry for the oversight, I added the dependency in version 11.2.1 and checked it was running fine, you should be able to update 🙂 |
|
Merci @StaloneLab ! J'ai mis la PR sur la bêta comme ça on peut tester directement sur les vrais contenus si les soucis rencontrés sont corrigés ou pas. |
|
Y'a du mieux ! Par exemple le PDF du billet de Gil Cot a été généré : https://beta.zestedesavoir.com/billets/pdf/4313/plaidoyer-pour-la-npi-et-la-gamme-woodstock.pdf Par contre celui de pierre sur le wifi a échoué, il manque encore quelques éléments spécifiques qui ont déjà généré deux tickets. |
|
Les deux contenus à tester prioritairement sont les billets 4313 et 4288 ? Plutôt que de faire cinq cents versions itérées, je peux envoyer localement les archives à ZMarkdown puis au compilateur LaTeX (sans tableaux parce qu'ils ne fonctionnent pas avec les nouvelles versions de LaTeX). |
|
J'ai réussi à compiler localement les deux contenus avec la version 11.2.2 venant d'être publiée. Pour le billet 4288, j'ai dû retirer deux images pour le faire compiler, j'espère que c'est une erreur locale (du fait que je n'ai pas les images par exemple), en tout cas ça ne semble pas lié aux échappements. Je vous laisse voir si ça fonctionne correctement sur le site. |
|
zmarkdown 11.2.2 est maintenant sur la bêta |
|
En testant, vérifier également que les maths dans le PDF de l'article https://zestedesavoir.com/articles/4445/demontrer-la-loi-de-snell-descartes/ sont correctement générées. Actuellement ce n'est pas le cas (le LaTeX généré est mauvais), mais peut-être que c'est corrigé avec cette version de zmd, puisque je ne reproduis pas le bug en testant sur https://zestedesavoir.github.io/zmarkdown/. |
|
Il y a une raison particulière pour que les dernières versions (11.2.2 et maintenant 11.2.3) ne soient pas déployées ou c'est un effet de calendrier des versions ? Je ne crois pas avoir eu de rapport d'erreurs liés à cette version en tout cas, il suffirait probablement de mettre à jour pour régler le problème mentionné à propos du contenu 4445. |
|
Actuellement la PR #6475, qui traite une des tâches confiées aux étudiants, est déployée sur la bêta. Je vais voir s'il est possible de déployer cette PR sur la bêta dans quelques jours avec la dernière version de zmarkdown pour vérifier si tous les soucis sont corrigés. |
|
J'ai essayé de faire le point en comparant les PDFs générés sur la prod et sur la bêta (avec cette PR déployée et en demandant le ré-export des contenus). Plaidoyer pour la NPI et la gamme WoodstockSur la prod il n'y a pas de PDF, sur la bêta on a maintenant un PDF, donc c'est forcément mieux. Problèmes :
Où placer son routeur Wi-Fi?Pas de PDF sur la prod, et l'export en PDF échoue toujours sur la bêta.
Il semble y avoir une erreur au niveau de la ligne 422 du fichier tex : \image{/opt/zds/data/contents-public/ou-placer-son-routeur-wi-fi__building/extra_contents/images/8V-AsW68Wk/RnyUk_oT02h.png}[Résultat pour un indice de réfraction de 1.2\textsuperscript{\footnotemark[1]}\footnotetext[1]{Qui déjà plus proche de celui de l’eau ou du \externalLink{bois}{https://link.springer.com/article/10.1007/s10762-019-00601-4}, si vous voulez un équivalent.} pour les murs, à comparer avec \externalLink{celui ci-dessus}{\#3-jouons-un-peu}. La réflexion est bien moins importante, et laisse sa place à la transmission.]Démontrer la loi de Snell-DescartesSur la prod il y a des problèmes dans les maths dans le PDF, problèmes corrigés dans le PDF généré sur la bêta. ConclusionQuelques petits bugs, mais j'ai l'impression qu'il n'y a pas de régression. |
|
Je jetterais un coup d’œil aux autres problèmes reportés, en particulier pour les images où remonter à la source devrait être assez simple, mais pour les tableaux, ça ne sera probablement pas corrigé dans l'instant : une refonte du fonctionnement des tableaux est en cours, et tout cela est chronophage. Néanmoins, c'est la priorité du moment pour ZMd, car les tableaux ne sont plus supportés par les versions récentes de LaTeX, donc il est possible d’espérer quelque chose à moyen terme. |
|
J'ai testé quelques contenus sur la bêta :
J'ai voulu tester en local de comparer l'export de PDF entre le tag v30.4 et la branche dev, mais je n'ai pas réussi à faire fonctionner la génération de PDF en local car TeXLive ne trouve pas le fichier |
|
Merci @Situphen pour les tests. J'ai regardé un peu plus en détails ce qui ne va pas avec les contenus pour lesquels l'export en PDF échoue. Un zeste de PythonJe ne suis pas sûr de moi pour celui-là. Ça d'abord commencé par un timeout (on limite la commande lualatex à 10 minutes). Ensuite ça s'exécutait rapidement (moins d'une minute), mais ce code Mardown (visible ici) : +-------+-------+-------+
| `'x'` | `' '` | `' '` |
+-------+-------+-------+
| `'o'` | `'o'` | `' '` |
+-------+-------+-------+
| `'x'` | `' '` | `' '` |
+-------+-------+-------+est converti en LaTeX : \begin{longtabu}{|m{\dimexpr(\linewidth) / 3 - 2 * \tabcolsep}|m{\dimexpr(\linewidth) / 3 - 2 * \tabcolsep}|m{\dimexpr(\linewidth) / 3 - 2 * \tabcolsep}|} \hline
\rowfont[l]{\bfseries}
\CodeInline{'x'} & \CodeInline{' '} & \CodeInline{' '} \\ \hline
\rowfont[l]{}
\CodeInline{'o'} & \CodeInline{'o'} & \CodeInline{' '} \\ \hline
\CodeInline{'x'} & \CodeInline{' '} & \CodeInline{' '} \\ \hline
\end{longtabu}et donne cette erreur à la compilation du PDF : Mais il semble qu'il y ait aussi une autre erreur :
(que l'on peut retrouver ici) et \begin{Verbatim}[commandchars=\\\{\}]
\PYGZpc{} python battle.py
debug None \PYG{o}{\PYGZob{}}\PYG{l+s+s1}{\PYGZsq{}pythachu\PYGZsq{}}: \PYG{o}{\PYGZob{}}\PYG{l+s+s1}{\PYGZsq{}name\PYGZsq{}}: \PYG{l+s+s1}{\PYGZsq{}Pythachu\PYGZsq{}}, \PYG{l+s+s1}{\PYGZsq{}attacks\PYGZsq{}}: \PYG{o}{[}\PYG{l+s+s1}{\PYGZsq{}charge\PYGZsq{}}, \PYG{l+s+s1}{\PYGZsq{}tonnerre\PYGZsq{}}, \PYG{l+s+s1}{\PYGZsq{}eclair\PYGZsq{}}\PYG{o}{]\PYGZcb{}}, \PYG{l+s+s1}{\PYGZsq{}pythard\PYGZsq{}}: \PYG{o}{\PYGZob{}}\PYG{l+s+s1}{\PYGZsq{}name\PYGZsq{}}: \PYG{l+s+s1}{\PYGZsq{}Pythard\PYGZsq{}}, \PYG{l+s+s1}{\PYGZsq{}attacks\PYGZsq{}}: \PYG{o}{[}\PYG{l+s+s1}{\PYGZsq{}charge\PYGZsq{}}, \PYG{l+s+s1}{\PYGZsq{}jet\PYGZhy{}de\PYGZhy{}flotte\PYGZsq{}}\PYG{o}{]\PYGZcb{}}, \PYG{l+s+s1}{\PYGZsq{}ponytha\PYGZsq{}}: \PYG{o}{\PYGZob{}}\PYG{l+s+s1}{\PYGZsq{}name\PYGZsq{}}: \PYG{l+s+s1}{\PYGZsq{}Ponytha\PYGZsq{}}, \PYG{l+s+s1}{\PYGZsq{}attacks\PYGZsq{}}: \PYG{o}{[}\PYG{l+s+s1}{\PYGZsq{}charge\PYGZsq{}}, \PYG{l+s+s1}{\PYGZsq{}jet\PYGZhy{}de\PYGZhy{}flamme\PYGZsq{}}\PYG{o}{]\PYGZcb{}\PYGZcb{}}
Valeur invalide
Monstre: \PYG{o}{(}pythachu/pythard/ponytha\PYG{o}{)}
\end{Verbatim}La programmation en C++ moderneCette équation en markdown (visible ici) : $$
\left\{\begin{aligned}
x(t) &= v_0 \,.\, cos \, \alpha \,.\, t \\
y(t) &= -g \,.\, \dfrac{t^2}2 + v_0 \,.\, sin \, \alpha \,.\, t
\end{aligned}\right.
$$est transformée en LaTeX : \[ \left\{\begin{aligned}
x(t) &= v_0 \,.\, cos \, \alpha \,.\, t \\
y(t) &= -g \,.\, \dfrac{t^2}2 + v_0 \,.\, sin \, \alpha \,.\, t
\end{aligned} \]Ce qui ne semble pas plaire à LaTeX : (il boucle sur cette erreur 100 fois et s'arrête car il est arrivé à 100 erreurs). Introduction à la preuve de programmes C avec Frama-C et son greffon WPLà, c'est vraiment bizarre. J'obtiens l'erreur suivante à la compilation du LaTeX en PDF : Alors qu'il n'y a que 5 balises Je n'ai pas essayé de reproduire ces erreurs dans mon environnement de dev pour mettre plus facilement ne évidence les bugs, je n'ai pas vraiment l'environnement qui va bien pour ça sous la main en ce moment. |
|
Hormis le problème des tableaux trop étroits, pour lequel nous n'avons pour le moment aucun correctif, du fait que la gestion des tableaux est en train d'être revue, je propose ci-dessous une synthèse quant aux différents problèmes reportés, en espérant n'en avoir oublié aucun. Problèmes liés aux imagesZMarkdown supporte correctement les images au format GIF, et je ne parviens pas à reproduire localement les deux erreurs reportées. Notes de bas de pageLe PDF est bien généré en utilisant le correctif présent sur ma branche Sur les échappementsLa commande Erreur de Zeste de PythonJe ne comprends pas l'erreur et ne suis pas en mesure de la reproduire localement du fait que j'ai une version de LaTeX trop récente (problème de tableaux...). Bloc caché inexistantJe note d'abord que le contenu contient une image GIF dont le lien est mort, on devrait le voir quand ZMarkdown reportera les avertissements à Sentry, il faudrait prévenir l'auteur (première image du contenu je crois, lien : http://frama-c.com/modern/framac.gif). SynthèseUne nouvelle version de ZMarkdown sera deployée bientôt, pour résoudre les problèmes identifiés. Prévenez-moi s'il faut que je teste plus en détail les erreurs LaTeX, je dois pouvoir me faire un VM de test avec la même version de LaTeX que celle présente sur le site. |
|
On fait quoi ici ? On attend encore une nouvelle version de zmd ? |
|
il me semble qu'on attendait le patch de sécurité, mais je sais plus où ça en était, c'est peut être déjà réglé. |
|
C'est effectivement réglé, cf. le courriel envoyé à la boite technique en date du 5 juillet. |
|
C'est fait ! |
There was a problem hiding this comment.
QA OK ✔️
Juste testé que poster un message formaté sur le forum fonctionne bien et qu'un export PDF d'un billet simple fonctionne.
L'objectif c'est surtout de rattraper le retard qu'on commençait un peu à accumuler par rapport à la version de zmd que zds-site utilisait.
N’hésitez pas à reporter les problèmes constatés tout de même, on ne peut pas indéfiniment accepter d’éventuelles régressions pour prétexte de minimiser le delta. |
Passage à zmarkdown 11.3.0
Ce qui est intéressant pour nous, c'est la correction d'un bug et le remplacement de la date d'exportation par la date de publication dans les PDF. Il y a l'option
enforce_shiftdont je n'ai pas vraiment compris le fonctionnement qui ne nous impacte pas actuellement mais nous impactera probablement dans la prochaine version majeure (car sa valeur par défaut va changer).Notes de version zmarkdown
QA :
upstream/devsource zdsenv/bin/activate && make zmd-start && make run-back)source zdsenv/bin/activate && python manage.py publication_watchdog)make zmd-stop) puis mettre à jour (make update) et relancer le site web (make zmd-start && make run-back)python manage.py publication_watchdog)admin