Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improve dissection process #2

Open
wants to merge 7 commits into
base: main
Choose a base branch
from
Open

Conversation

Prometheus3375
Copy link
Owner

@Prometheus3375 Prometheus3375 commented Jan 3, 2025

Resolves #1.

TODO

  • Implement move analyzer to verbosely describe moves made and generate appropriate steps.
  • Implement method StateWithAllPositions.yield_solutions which is similar to solve, but it yields all done states with the least number of moves.
    • Method solve instead analyzes all solutions, takes the one with least number of steps and returns it with the steps.
  • Add names for players in the main room. Distinguish between dissection player and ad clear ones.
    • Include these names in dissection steps similar to pass steps.
  • Add tests for multiset?

@Prometheus3375 Prometheus3375 self-assigned this Jan 3, 2025
Copy link
Owner Author

@Prometheus3375 Prometheus3375 left a comment

Choose a reason for hiding this comment

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

Progress so far.

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.

Does not work with all given combinations
1 participant