Skip to content

43: centered the pattern - #44

Merged
OnixTech merged 1 commit into
mainfrom
43-bug-06-pattern-not-centered
Aug 23, 2026
Merged

43: centered the pattern#44
OnixTech merged 1 commit into
mainfrom
43-bug-06-pattern-not-centered

Conversation

@semere-meb

Copy link
Copy Markdown
Member

Summary

Puts the pattern at center correctly.

Fixes #43

Changes

  • MazeGenerator._put_pattern now correctly calculates the offset avoiding double floor divisions
  • MazeGenerator._put_pattern x,y format fixed.

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 23, 2026 17:56
@semere-meb semere-meb self-assigned this Aug 23, 2026
@semere-meb semere-meb linked an issue Aug 23, 2026 that may be closed by this pull request
@semere-meb semere-meb changed the title centered the pattern 43: centered the pattern Aug 23, 2026
@OnixTech
OnixTech merged commit fedca79 into main Aug 23, 2026
2 checks passed
@semere-meb
semere-meb deleted the 43-bug-06-pattern-not-centered branch August 24, 2026 08:49
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.

BUG-06: pattern not centered

2 participants