feat(installeur): un rôle par sortie adressable, et des toys enfin nommés - #284
Merged
Merged
Conversation
…mmés Lot 2 des toys. Le nom du toy dans cabinet.xml était générique : « Backboard HD » pour la première matrice, « Matrice 2 », « Ruban 1.2 »… Exact, mais illisible le jour où l'on relit son cabinet — et c'est ce nom que DOF cherche ensuite pour piloter la sortie. Chaque sortie reçoit maintenant un rôle : backboard, sous le meuble, côté gauche, côté droit, fronton, flippers, intérieur du meuble. En mode matrice, un rôle pour tout le contrôleur ; en mode rubans, un par sortie utilisée, avec le nombre de LED affiché à côté pour qu'on sache de quoi on parle. Le rôle reste FACULTATIF. Sans lui, le nom générique demeure : rien ne change pour un cabinet déjà configuré, et la proposition n'en avance aucun — cohérent avec PINCABOS_TOYS_SANS_SUPPOSITION_V1. « backboard » garde volontairement « Backboard HD » : c'est le nom qu'ont les cabinets en service, et le changer casserait leur configuration DOF. Les cinq langues suivent. 765 tests. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Lot 2 des toys. S'empile sur #283 (étape Boutons) : les deux touchent le gabarit et les libellés.
Le problème
Le nom du toy dans
cabinet.xmlétait générique :Exact, mais illisible le jour où l'on relit son cabinet — et c'est ce nom que DOF cherche ensuite pour piloter la sortie.
Ce que ça donne
Sept rôles : backboard, sous le meuble, côté gauche, côté droit, fronton, flippers, intérieur du meuble. En mode matrice, un rôle pour tout le contrôleur ; en mode rubans, un par sortie utilisée — avec le nombre de LED affiché à côté, pour savoir de quoi on parle.
Et le
cabinet.xmlsort avec :Deux garanties de non-régression
Le rôle est facultatif. Sans lui, le nom générique demeure — rien ne change pour un cabinet déjà configuré. Et la proposition n'en avance aucun, cohérent avec
PINCABOS_TOYS_SANS_SUPPOSITION_V1de #282 : on ne décrit pas le cabinet à la place de son propriétaire.backboardgarde « Backboard HD ». C'est le nom qu'ont les cabinets en service ; le changer casserait leur configuration DOF. Un test le verrouille.Tests
test_toys_roles.py, 13 tests : le catalogue, la compatibilité du nom historique, un rôle inconnu refusé, le rôle facultatif, la proposition qui n'en avance aucun, les toys nommés en rubans comme en matrice, une sortie vide qui ne produit pas de toy, et un libellé par rôle dans les cinq langues.765 tests.
Suite
Lot 3 : le catalogue DOF complet — contacteurs, gyrophare, shaker, knocker, ventilateur — chacun avec son type DOF, sa carte et sa sortie.
🤖 Generated with Claude Code