Skip to content

fix(Sudoku): add convention prints #1946 to 3 .NET notebooks#1964

Closed
jsboige wants to merge 1 commit into
mainfrom
fix/sudoku-search-dotnet-convention-print-1946
Closed

fix(Sudoku): add convention prints #1946 to 3 .NET notebooks#1964
jsboige wants to merge 1 commit into
mainfrom
fix/sudoku-search-dotnet-convention-print-1946

Conversation

@jsboige
Copy link
Copy Markdown
Owner

@jsboige jsboige commented May 31, 2026

Summary

Convention #1946: add informative Console.WriteLine to silent code cells in Sudoku .NET notebooks.

Changes

Notebook Cell Type Change
Sudoku-1-Backtracking-Csharp BacktrackingDotNetSolver class def C Console.WriteLine("Classe BacktrackingDotNetSolver definie")
Sudoku-11-Choco-Csharp IKVM/Choco DLL loading A Console.WriteLine("DLLs IKVM + Choco-solver chargees")
Sudoku-14-BDD-Csharp BDD exercise stub B Console.WriteLine("Exercice a completer : solveur BDD avec propagation de contraintes")

Execution proof

  • Papermill .net-csharp: 3 notebooks, 26 code cells, 0 errors
  • All cells have execution_count != null and outputs != []
  • Insertions > deletions (2449 insertions, 924 deletions = re-execution outputs)

Notable exclusion

GeneticSharp-EdgeDetection exercise stub was excluded: adding Console.WriteLine at the end of a cell that contains class definitions causes .NET Interactive compilation failure. This is a kernel limitation, not a notebook bug.

Catalog

Regenerated: COURSE_CATALOG.generated.json (504 entries)

Co-Authored-By: Claude Opus 4.8 noreply@anthropic.com

…ion informative output)

- Sudoku-1: Console.WriteLine after BacktrackingDotNetSolver class def
- Sudoku-11: Console.WriteLine after IKVM/Choco DLL loading
- Sudoku-14: Console.WriteLine in BDD exercise stub
- Re-executed with Papermill .net-csharp: 26 cells, 0 errors
- Catalog regenerated (504 entries)

Note: GeneticSharp-EdgeDetection excluded (exercise stub with class def
causes .NET Interactive compilation error when Console.WriteLine added
at end of cell with class definitions).

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@github-actions
Copy link
Copy Markdown

Notebook PR Validation: PASS

  • Notebooks checked: 3
  • Code cells validated: 25
  • Result: All passed

Checks: H.1 (no errors), H.3 (execution_count), C.1 (no banned patterns)
Non-Python kernels (.NET/Lean): C.1 + errors only (execution_count advisory)
QuantConnect notebooks: C.1 + errors only (require QC Cloud for execution)

Copy link
Copy Markdown
Collaborator

@clusterManager-Myia clusterManager-Myia left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[Hermes] — APPROVED

Convention prints sur 3 notebooks .NET + mise à jour COURSE_CATALOG.generated.json. Promotions BETA->PRODUCTION cohérentes avec ajout d'outputs. Cell count adjustments logiques (with_outputs down, without_outputs to 0). Risque nul.

@jsboige
Copy link
Copy Markdown
Owner Author

jsboige commented Jun 1, 2026

Superseded by #2006 (conclusions + catalog fix) and #2018 (outputless fix + re-exec + IP cleanup, merged).

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants