Skip to content

Conversation

benwh1
Copy link

@benwh1 benwh1 commented Aug 22, 2024

Add better 4x4x4 definition with pieces in a logical order and all block moves and rotations defined

@rokicki
Copy link
Collaborator

rokicki commented Aug 23, 2024

I agree this is better in some ways, but not all. Primarily, at the moment, a .tws file encompasses not only a puzzle definition, but also implicitly a metric (since twsearch has a default move set of all non-rotation moves).

I definitely see the utility of having a .tws file that includes all moves, but I'd like to get there in stages. First, I want twsearch to support composition of moves in move definitions (to make the file sizes reasonable). Second, I'd like .tws to include metric definitions that can then be chosen by the user. I'd like cubing.js to generate both of these automatically, so all puzzle-geometry-based puzzles just get a "default" .tws file generated by puzzle geometry, and not anything manually manipulated.

At the moment, the existing puzzle definitions are good default definitions, as they provide a reasonable default metric (and for 444 we have both obtm and slice turn versions of the files).

So at the moment I'm leaning against integrating this pull request. But I encourage discussion.

@benwh1
Copy link
Author

benwh1 commented Aug 23, 2024

The problem with most of the current definition files is that they don't say what order the pieces are in, which makes them almost useless if you want to actually type in an explicit position and solve it.

@rokicki
Copy link
Collaborator

rokicki commented Aug 23, 2024 via email

@lgarron lgarron force-pushed the main branch 8 times, most recently from bc46645 to 4690eef Compare October 9, 2024 00:34
@lgarron lgarron force-pushed the main branch 5 times, most recently from a9a6719 to 5f57024 Compare December 1, 2024 07:34
@lgarron lgarron force-pushed the main branch 5 times, most recently from 0e4717d to 83dca4b Compare January 10, 2025 14:04
@lgarron lgarron force-pushed the main branch 6 times, most recently from 80e28f3 to d034656 Compare March 12, 2025 23:50
@lgarron lgarron force-pushed the main branch 4 times, most recently from 8d51e28 to 3b30829 Compare April 5, 2025 02:03
@lgarron lgarron force-pushed the main branch 4 times, most recently from c1f5bcb to e7cd1fa Compare April 10, 2025 13:20
@lgarron lgarron force-pushed the main branch 2 times, most recently from b6b8f9a to e726135 Compare May 6, 2025 07:44
@lgarron lgarron force-pushed the main branch 2 times, most recently from 3e30b1b to 34d42d9 Compare July 16, 2025 02:41
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.

2 participants