Skip to content

added sokoban puzzle#10

Open
D-W-Chiang wants to merge 17 commits intomainfrom
sp26-sokoban
Open

added sokoban puzzle#10
D-W-Chiang wants to merge 17 commits intomainfrom
sp26-sokoban

Conversation

@D-W-Chiang
Copy link
Copy Markdown

couple of notes:

  • condensed hashing (only stores position of the player plus positions of the boxes)
  • generate moves returns a list that contains only moves that involve pushing a box

@JesterCoding JesterCoding requested a review from AEstrellaS April 23, 2026 04:38
@JesterCoding
Copy link
Copy Markdown

Add sokoban puzzle.

There is a small bug on the front-end on the UWAPI side with arrow length size. Please fix!

@JesterCoding JesterCoding changed the title added sokoban file added sokoban puzzle Apr 23, 2026
Comment thread solver/src/solver/solver.py
@JesterCoding JesterCoding requested a review from AEstrellaS April 24, 2026 22:34
@JesterCoding
Copy link
Copy Markdown

I have changed the file back to what it previously was. It looks like it is not necessary for the solver.py to have the particular instance there with Abraham's changes.

@hpghsu1314
Copy link
Copy Markdown
Contributor

Why is venv in the .gitignore? Is there a venv file somewhere? That should not be the case.

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.

4 participants