fix(inovmicro-exao): Ajout des photos manquantes#175
Conversation
|
Images automagically compressed by Calibre's image-actions ✨ Compression reduced images by 23.5%, saving 153.8 KB.
|
- Alt text des deux <figure> ne mentionnait plus le sujet de la fiche (« Thermomètre très lisible sur la STeaMi », copié-collé d'une autre fiche). Remplacement par une description fidèle de chaque photo. - Figcaption « via l'accelerometre » → « via l'accéléromètre » (texte affiché, donc accents requis). Refs #162
nedseb
left a comment
nedseb
left a comment
There was a problem hiding this comment.
Merci pour les photos Mattéo, elles sont nettes et illustrent bien le résultat sur la carte. Quelques points à revoir avant merge ; je viens de pousser un commit (65192dc) qui adresse les deux plus évidents (alt text + accent figcaption), il reste les sujets pédagogiques ci-dessous :
Bloquant : régression d'API dans afficher_capteur
Le code passe de :
ecran.title(orientation)
ecran.text("X:...g", at="NW")
ecran.text("Y:...g", at="W")
ecran.text("Z:...g", at="SW")à :
ecran._d.text(orientation, 25, 30, 15) # x=10, y=10, couleur=blanc fixe
ecran.value("{:+.2f}".format(acceleration_x), label="X", unit="g")
ecran.subtitle(
\"Y:{:+.2f}g\".format(acceleration_y),
\"Z:{:+.2f}g\".format(acceleration_z),
)Trois soucis cumulés ici :
ecran._d.text()est un accès direct au driver bas niveau (le préfixe_dsigne une API privée). Le projet construit le wiki autour de l'API haut niveausteami_screen: les autres fiches qui ont une mise en page similaire (i15, i11, i10) restent dans cette API, et l'encart « API bas niveau » d'i10 oriente versscreen.pixel()/line()/rect()/circle()quand on a besoin de coordonnées exactes — pas vers_d.text. Si l'objectif est juste de rendre le titre en blanc plutôt qu'enGRAY, le canal idiomatique serait plutôtecran.text(orientation, at=\"N\", color=WHITE)(cf. couleurs documentées dans i10).- Le commentaire ment : il annonce
x=10, y=10, couleur=blanc fixealors que les arguments réels sont25, 30, 15. Soit on le supprime, soit on aligne les valeurs. - Coordonnées magiques (
25, 30, 15) sans nommage — contraire à l'esprit « identifiants/constantes parlants » des conventions MicroPython STeaMi.
Le ecran.subtitle(\"Y:...\", \"Z:...\") en deux arguments est OK, c'est le même pattern que i15.
Description PR à corriger
Corrige le commentaire inline de
ecran._d.text()qui indiquait des coordonnées incorrectes
Sur main, ecran._d.text() n'existe pas du tout : cette PR introduit le code (avec son commentaire faux). La description gagnerait à dire « refonte de l'affichage de afficher_capteur pour utiliser value()/subtitle() comme i15 » plutôt que « correction d'un commentaire ».
Cases du template sur-cochées
Les cases « Catalogue » et « Code (composant React, page, type, CSS) » sont cochées dans la PR mais seuls la fiche markdown et les deux JPEG sont touchés. À décocher pour la lisibilité.
Mineur : nommage des fichiers
Steami_prog1.jpeg / Steami_amelioration3.jpeg mélangent CamelCase et snake_case. Le wiki utilise soit un préfixe numéroté kebab-case (i10 : 01-steami-oled-vide.jpeg...), soit un nom descriptif simple en snake_case (i15 : steami_graph.jpeg). Idéalement à renommer en 01-prog-principal.jpeg / 02-amelioration-niveau-a-bulle.jpeg (ou équivalent kebab-case) pour rester homogène avec i10 qui a aussi plusieurs photos. Cosmétique, mais autant le faire maintenant.
Périmètre de la PR
Le titre annonce « Ajout des photos manquantes » mais la PR refait aussi l'affichage du programme principal — typiquement deux PRs distinctes. Pas bloquant vu la taille, juste à garder en tête pour la prochaine fois.
There was a problem hiding this comment.
Pull request overview
Cette PR complète la fiche I‑NOVMICRO EXAO i09-inclinaison-accelerometre en ajoutant des visuels réels de la carte STeaMi en fonctionnement, afin de corriger le manque de photos “en situation” signalé dans #162.
Changes:
- Ajout de deux
<figure>(photo du programme principal + photo de l’amélioration “niveau à bulle”) dans la fiche i09. - Mise à jour du code d’affichage OLED présenté dans la fiche (orientation + valeurs).
- Correction/alignement des légendes (
figcaption) pour décrire fidèlement le contenu des photos.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| ecran.text("X:{:+.2f}g".format(acceleration_x), at="NW") | ||
| ecran.text("Y:{:+.2f}g".format(acceleration_y), at="W") | ||
| ecran.text("Z:{:+.2f}g".format(acceleration_z), at="SW") | ||
| ecran._d.text(orientation, 25, 30, 15) # x=10, y=10, couleur=blanc fixe |
|
rtMattéo, petit point de coordination sur cette PR : État actuelDepuis l'ouverture, 9 PRs ont mergé sur Il faut donc :
Tu peux faire : git fetch origin main
git rebase origin/main
# Résoudre les conflits éventuels, ne garder que les ajouts photos + ton commit d'origine
git push --force-with-leasePoint bloquant de ma revue (toujours non adressé)Ma ecran._d.text(orientation, 25, 30, 15) # API privée + commentaire qui ne correspond pas aux argumentsIdéalement, remplacer par l'API publique En attendantOn laisse cette PR de côté pour l'instant. Quand tu auras rebasé + ajusté le code, fais signe et on la merge rapidement. |
Résumé
i09-inclinaison-accelerometre: une photo du programme principal (orientation + valeurs X/Y/Z sur l'écran OLED) et une photo de l'amélioration niveau à bulle (barre de progression inclinaison)figcaptionqui contenaient un texte générique erroné ("température et commentaire de confort") remplacé par des descriptions fidèles au contenu de chaque photoecran._d.text()qui indiquait des coordonnées incorrectesType de changement
resources.tsouprojects.tsTest plan
npm run build(danssite/) passe sans erreur ni nouveau warningnpm run typecheckpasse<figure>, affichage des photos, lisibilité desfigcaption)npm run format:check && npm run lint:mdCLAUDE.mdIssues liées
Closes #162
Refs #137 (PR d'origine de la fiche i09)