Skip to content

45: bonus 02 support multiple maze generation algorithms - #74

Merged
OnixTech merged 2 commits into
masterfrom
45-bonus-02-support-multiple-maze-generation-algorithms
Aug 29, 2026
Merged

OnixTech merged 2 commits into
masterfrom
45-bonus-02-support-multiple-maze-generation-algorithms

Conversation

@semere-meb

Copy link
Copy Markdown
Member

Summary

Adds animation support by showing each cell's wall as it gets carved out.

Fixes #46

Changes

  • renamed MazeGenerator.render_maze -> MazeGenerator.paint_walls
  • new MazeGenerator.carve_walls to remove the walls shared between 2 neighbors
  • wrapped mlx.mlx_put_image_to_window with MazeGenerator.put_image to run it multiple times ensuring immediate effect
  • updated flake.nix to pass through python 3.10, so the project uses python 3.10 for the project

Testing

  • make lint passes locally
  • make test passes locally
  • Manually tested the behavior described above

Checklist

  • PR title follows <TICKET-ID>: short description
  • Linked to the correct issue (Fixes #... above)
  • No unrelated changes bundled into this diff
  • Updated README if this changes user-facing behavior (config keys, controls, etc.)

@semere-meb
semere-meb requested a review from OnixTech August 29, 2026 14:23
@semere-meb semere-meb self-assigned this Aug 29, 2026
@semere-meb
semere-meb force-pushed the 45-bonus-02-support-multiple-maze-generation-algorithms branch from 811be1b to 2f2b546 Compare August 29, 2026 14:34

@OnixTech OnixTech left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Great

@OnixTech
OnixTech merged commit e3f7596 into master Aug 29, 2026
2 checks passed
@semere-meb
semere-meb deleted the 45-bonus-02-support-multiple-maze-generation-algorithms branch August 29, 2026 14:55
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.

BONUS-03: Animation during maze generation

2 participants