fix(search): guard exercise cells to prevent execution errors#291
Merged
fix(search): guard exercise cells to prevent execution errors#291
Conversation
Search-1: Wrap bfs_solve(mc) and visualize_maze() in try/except to handle NameError/NotImplementedError when exercises are incomplete. Search-8: Comment out raise NotImplementedError in exercise 1 stub to allow full notebook execution without student code. Ensures all 11 Search notebooks pass automated execution for EPITA TP. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
Clear outputs and execution_count from 43 exercise cells in 17 notebooks (Part1-Foundations, Part2-CSP, Applications). Exercise cells should have no pre-filled outputs to prevent execution errors and keep solutions empty for students. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
jsboige
commented
Apr 19, 2026
Owner
Author
jsboige
left a comment
jsboige
left a comment
There was a problem hiding this comment.
Review ClusterManager — LGTM. Nettoyage correct des outputs des cellules exercices dans les notebooks Search (execution_count: null + outputs supprimés). Prévient la divulgation des solutions aux étudiants. Approche standard pour les notebooks pédagogiques. ✅
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.
Summary
bfs_solve(mc)andvisualize_maze(maze, solution)in try/except to handle NameError when students haven't completed the MissionariesCannibals classraise NotImplementedError("A completer")stub that blocked full notebook executionTest plan
notebook_tools.py executeContext
Preparation for EPITA TP Exploration (Search) on Monday 20/04 morning.
Co-Authored-By: Claude Opus 4.7 noreply@anthropic.com