Skip to content

MGS : 9 cellules d'exercice sont des exemples résolus étiquetés 'exercice' (MGS-30 : 3/3) #13515

Closed

Description

@myia-ai-01

La remarque

Sur PR #13397 (2026-08-28T19:01Z), mergée 6 h 27 plus tard sans réponse :

Les exercices ne sont ils pas essentiellement corrigés?? Ou j'ai mal compris peut-être.

Vous aviez raison. Vérifié firsthand sur main.

Mesure

MGS-30-ScatterSearch-Decomposition.ipynb — les 3 cellules d'exercice portent un // TODO etudiant mais le code sous le commentaire est entièrement écrit :

cellule exercice lignes de code effectif état
16 Exercice 1 — budget ×4 14 résolu
18 Exercice 2 — balayage QualityFraction 14 résolu
20 Exercice 3 — profil decode vs cost 35 résolu

L'exercice 1 dit « remplace resultats64 par les courses » — or resultats64 est déjà rempli par la boucle qui suit, et la sortie committée affiche les quatre graines avec leurs deltas. Seule la phrase de verdict reste à écrire.

Ce sont donc des exemples résolus étiquetés « exercice » — le défaut que exercise-example-labeling.md demande de trancher par le contenu, jamais par l'étiquette.

Ampleur — bornée, mesurée sur les 29 notebooks MGS

TOTAL                                        96 exos    9 pre-resolus   (9%)

Concentration nette :

notebook exos pré-résolus
MGS-30-ScatterSearch-Decomposition 3 3
MGS-7b-LandscapeMultidim 3 2
MGS-16 / MGS-17 / MGS-21 / MGS-25 4-5 1 chacun

Les 23 autres notebooks MGS sont sains (0 pré-résolu). Le défaut n'est pas systémique : il est concentré, et MGS-30 — le notebook sur lequel la remarque a été postée — est le pire cas.

Travail attendu

Par cellule concernée, trancher par le contenu :

  • soit rendre l'exercice réel : retirer le corps calculé, laisser un stub conforme C.1 (pass / print("Exercice a completer") / resultats = None # TODO etudiant) en conservant les # TODO / # Indice / # Etape N — jamais raise NotImplementedError ;
  • soit assumer l'exemple : renommer en « Exemple résolu » et ajouter un vrai exercice à côté, pour tenir les trois exercices par notebook.

Priorité à MGS-30 (3/3) puis MGS-7b (2/3).

Contrainte

Toute cellule code modifiée est ré-exécutée avant commit (C.2) ; le notebook doit s'exécuter de bout en bout exercices non complétés (C.1).

See #12373

Activity

  1. jsboige commented on Aug 29, 2026

    @jsboige
    Owner

    [CLAIMED] #13515 — myia-po-2026:CoursIA 2026-08-29T17:10Z — paths: MyIA.AI.Notebooks/Search/Part4-Metaheuristics/MGS-30-ScatterSearch-Decomposition.ipynb MyIA.AI.Notebooks/Search/Part4-Metaheuristics/MGS-7b-LandscapeMultidim.ipynb (+ MGS-16/17/21/25 si fenetre)

  2. jsboige commented on Aug 29, 2026

    @jsboige
    Owner

    [DELIVERED PARTIEL] #13515 — myia-po-2026:CoursIA 2026-08-29T18:20Z — PR #13525 (paths: MGS-30-ScatterSearch-Decomposition.ipynb MGS-7b-LandscapeMultidim.ipynb README.md). 5/9 cellules (priorites MGS-30 3/3 + MGS-7b 2/2), re-exec reelle 10/10 et 8/8. Residuel : singletons MGS-16/17/21/25.

  3. added a commit that references this issue on Aug 29, 2026
  4. jsboige commented on Aug 29, 2026

    @jsboige
    Owner

    [DELIVERED TRANCHES 1+2] #13515 — myia-po-2026:CoursIA 2026-08-29T19:05Z — Tranche 1 : PR #13525 (MGS-30 x3 + MGS-7b x2, 5/9). Tranche 2 : PR #13526 — MGS-21 cell 9 relabel Exemple resolu (option B : le vrai Exercice 3 existe en section 3, markdown cell 10 ancre ses chiffres sur la sortie). Reconciliation du census (verifie firsthand) : MGS-16 c27 / MGS-17 c17-23 / MGS-25 c14-18 = stubs sains (FP du census — squelettes types / solutions commentees). 0 pre-resolu reel restant apres ces 2 PR. paths: MyIA.AI.Notebooks/Search/Part4-Metaheuristics/MGS-21-Representation-vs-Algorithme.ipynb

  5. added a commit that references this issue on Aug 29, 2026
  6. added a commit that references this issue on Aug 29, 2026
  7. added 3 commits that reference this issue on Aug 30, 2026
  8. added
    candidate-deliveredReferenced by a merged PR with no post-merge activity -- candidate for close triage (#10466)
    on Aug 31, 2026
  9. jsboige commented on Aug 31, 2026

    @jsboige
    Owner

    [TRIAGE candidate-delivered] Acceptation satisfaite firsthand — prête à fermer par le coordinateur.

    Vérification sur origin/main, après lecture du body, de tous les commentaires et des PRs #13525/#13526 (bodies, reviews, commentaires, diffs) :

    • fix(search,#13515): de-leak MGS-30 (3/3) + MGS-7b (2/3) — exercices pré-résolus stubbés #13525 (MERGED) rend réels les 5 exercices prioritaires : MGS-30 cellules 16/18/20 et MGS-7b cellules 11/12 sont désormais des squelettes TODO conformes C.1. Exécution réelle attestée et encore présente sur main : MGS-30 10/10 cellules code exécutées, 0 erreur ; MGS-7b 8/8, 0 erreur.
    • fix(search,#13515): MGS-21 — la démonstration étiquetée « Exercice 3 » devient « Exemple résolu » (tranche 2) #13526 (MERGED) tranche MGS-21 cellule 9 par le contenu : le corps calculé est correctement renommé « Exemple résolu », tandis que le vrai Exercice 3 reste un stub distinct. Exécution 9/9, 0 erreur ; les deux réserves Hermes ont été corrigées puis levées sur le dernier head avant merge.
    • Réconciliation firsthand des trois singletons restants :
      • MGS-16 c27 : classe squelette avec return 0.0 // TODO étudiant et message « à compléter », pas une solution ; notebook 14/14 exécuté, 0 erreur.
      • MGS-17 c17/c19/c21/c23 : quatre stubs TODO (0.0/message « à compléter »), pas de corps résolu ; notebook 11/11 exécuté, 0 erreur.
      • MGS-25 c14/c16/c18 : solutions entièrement commentées sous l'instruction « Décommentez et exécutez », sortie active limitée au message « exercice à compléter » ; notebook 9/9 exécuté, 0 erreur.

    Conclusion : les 6 pré-résolus réels du body sont traités (5 stubbés + 1 assumé comme exemple) ; les hits restants du census sont des faux positifs de classification, pas une dette pédagogique. Aucun résiduel réel trouvé.

  10. jsboige commented on Sep 1, 2026

    @jsboige
    Owner

    [INFO delivered-urn — fermeture] Vérification FIRSTHAND concordante avec le triage du 2026-08-31T15:33Z : les 2 PRs livrantes sont MERGED (gh pr view à l'instant) — #13525 (2026-08-30T16:50Z, MGS-30 ×3 + MGS-7b ×2 stubbés, re-exec 10/10 + 8/8) et #13526 (2026-08-29T22:21Z, MGS-21 relabel « Exemple résolu »). Acceptance du body satisfaite : les 9 cellules pré-résolues sont traitées (5 stubbées conformes C.1 + 1 relabelée exemple + census réconcilié — MGS-16/17/25 = stubs sains, FP du census, vérifié par la tranche 2). 0 pré-résolu réel restant. Fermeture par protocole delivered-urn.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    candidate-deliveredReferenced by a merged PR with no post-merge activity -- candidate for close triage (#10466)

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions