OyvindSabo/quoridor.js issues and pull requests

#54 - getWinner is not documented in README

Issue - State: open - Opened by OyvindSabo over 1 year ago

#53 - isValidMove returned true for invalid move

Issue - State: closed - Opened by OyvindSabo over 1 year ago

#52 - Create mirrorHashVertically

Issue - State: open - Opened by OyvindSabo about 2 years ago

#51 - Create mirrorHashHorizontally

Issue - State: closed - Opened by OyvindSabo about 2 years ago

#50 - Expose getHash function

Issue - State: closed - Opened by OyvindSabo about 2 years ago

#49 - No need to maintain both board and wallMatrix and pieceMatrix

Issue - State: closed - Opened by OyvindSabo about 2 years ago

#48 - Improve performance of getUnicodeRepresentation

Issue - State: closed - Opened by OyvindSabo about 2 years ago

#47 - pieceMatrix and wallMatrix should be maps from position to player instead of from x to y to player

Issue - State: closed - Opened by OyvindSabo about 2 years ago

#46 - All constants should be upper cased with underscores between the words

Issue - State: open - Opened by OyvindSabo about 2 years ago

#45 - Store playerPositions as PawnPosition strings instead of PiecePosition objects

Issue - State: closed - Opened by OyvindSabo over 2 years ago

#44 - Improve performance of makeUnvalidatedMove

Issue - State: open - Opened by OyvindSabo over 2 years ago

#43 - Make it possible to get the board state as a bit board that can be easily mirrored, for instance

Issue - State: open - Opened by OyvindSabo over 2 years ago

#42 - Add a simple way to mirror the board, or at least the board hash

Issue - State: open - Opened by OyvindSabo over 2 years ago

#41 - Reimplement a* more efficiently

Issue - State: closed - Opened by OyvindSabo over 2 years ago

#40 - getValidMove array should place wall moves that block the shortest path before other wall moves

Issue - State: open - Opened by OyvindSabo over 2 years ago

#39 - isMove should use a map instead of looping through all moves

Issue - State: open - Opened by OyvindSabo over 2 years ago

#38 - Use maps instead of switches for better performance

Issue - State: closed - Opened by OyvindSabo over 2 years ago

#37 - Document isMoveInteresting in README

Issue - State: open - Opened by OyvindSabo over 2 years ago

#36 - Rename shortestPath to getShortestPath

Issue - State: closed - Opened by OyvindSabo over 2 years ago

#35 - Create and export isMoveInteresting function

Issue - State: closed - Opened by OyvindSabo over 2 years ago

#34 - isValidMove returns false for valid move

Issue - State: open - Opened by OyvindSabo over 2 years ago

#33 - Expose a function called getShortestPathLength

Issue - State: closed - Opened by OyvindSabo over 2 years ago - 1 comment

#32 - Consolidate inconsistent naming of move / position

Issue - State: open - Opened by OyvindSabo over 2 years ago

#31 - No need to run a star if only vertical walls have been placed

Issue - State: closed - Opened by OyvindSabo over 2 years ago

#30 - Avoid running path validation if the newly placed wall does not touch two other walls or one other wall and the edge of the board

Issue - State: closed - Opened by OyvindSabo over 2 years ago

#29 - Document makeUnvalidatedMove in README

Issue - State: open - Opened by OyvindSabo over 2 years ago

#28 - Document makeMove in README

Issue - State: open - Opened by OyvindSabo over 2 years ago

#27 - Expose makeUnvalidatedMove function

Issue - State: closed - Opened by OyvindSabo over 2 years ago

#26 - The first wall move will always be valid, so no need to run validation in this case

Issue - State: closed - Opened by OyvindSabo over 2 years ago

#25 - When finding valid wall moves don't run shortest path algorithm if all previous walls are the same direction of the wall being evaluated

Issue - State: closed - Opened by OyvindSabo over 2 years ago

#24 - Get rid of all use of moveObjectToMove

Issue - State: closed - Opened by OyvindSabo over 2 years ago

#23 - Diagonal jump agains wall not returned by getValidMoveArray

Issue - State: closed - Opened by OyvindSabo over 2 years ago

#22 - Uncaught TypeError: Cannot read properties of undefined (reading '6')

Issue - State: closed - Opened by OyvindSabo over 2 years ago

#21 - Further optimizations to getValidWallMoveArray

Issue - State: open - Opened by OyvindSabo over 2 years ago

#20 - Make getValidWallMoveArray faster by not checking shortest path for every wall

Issue - State: closed - Opened by OyvindSabo over 2 years ago

#19 - Error: a cannot be decremented.

Issue - State: closed - Opened by OyvindSabo over 2 years ago

#18 - Horizontal row not visualized between eight and ninth row

Issue - State: open - Opened by OyvindSabo over 2 years ago

#17 - Add isMove function that can be used as a type guard to convert a string to a Move

Issue - State: closed - Opened by OyvindSabo over 2 years ago

#16 - It is possible to trap both yourself and the other player

Issue - State: closed - Opened by OyvindSabo over 2 years ago

#15 - It should be possible to provide timestamps to moves

Issue - State: open - Opened by OyvindSabo over 2 years ago

#14 - There should be a way to get the minimal data required to reproduce the game state

Issue - State: open - Opened by OyvindSabo over 2 years ago - 1 comment

#13 - Add isMove function

Issue - State: open - Opened by OyvindSabo over 2 years ago

#12 - Make it possible to import the types as well.

Issue - State: open - Opened by OyvindSabo over 2 years ago

#11 - Remove unused dependencies

Issue - State: open - Opened by OyvindSabo over 3 years ago

#10 - Perhaps make it possible to redo moves

Issue - State: closed - Opened by OyvindSabo over 3 years ago

#9 - Rename getAsciiRepresentation to getUnicodeRepresentation

Issue - State: closed - Opened by OyvindSabo over 3 years ago

#8 - Rename isValidMove to isMoveValid

Issue - State: closed - Opened by OyvindSabo over 3 years ago

#7 - The logic in isDoubleUpMove does not make sense.

Issue - State: closed - Opened by OyvindSabo over 3 years ago

#6 - Rename getHorizontalCoordinate and getVerticalCoordinate to getXCoordinate and getYCoordinate

Issue - State: open - Opened by OyvindSabo over 3 years ago
Labels: refactor

#5 - Optimizing legal move generation

Issue - State: open - Opened by alfredbirk over 3 years ago - 2 comments

#4 - Add getTurn function

Issue - State: closed - Opened by OyvindSabo over 3 years ago

#3 - Add undoLastMove function

Issue - State: closed - Opened by OyvindSabo over 3 years ago

#2 - History should be of type Move[]

Issue - State: closed - Opened by OyvindSabo over 3 years ago

#1 - npm run lint does not warn about unused imports

Issue - State: open - Opened by OyvindSabo over 3 years ago

GitHub / OyvindSabo/quoridor.js issues and pull requests