twisty_puzzles 0.0.9 → 0.0.10

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 3dfb23949825572c531ac7bfcea491151629f8673160605069b1793a827d961a
-  data.tar.gz: f343cc589e99ac129717f47a7b3dbe4dcb3d4a0cbcae2e0f8c42b12b22ad842c
+  metadata.gz: 7616a5e5bd64a5beb6ea98bed6a6f6bc2dc5ae145491db533e4693cd10d0ff6d
+  data.tar.gz: 8bc2cd597844fb86f9168c8e368c6bbb01cc34a82598ebef7f5dcb77c1e3713c
 SHA512:
-  metadata.gz: 2fc20aef94e333318c53ee292b5418424ad8880cc2f8ff18c2102d0799d96728ccc9a98869269bf9c6a046174dcf5f71182cf6498021838d8772c18a7008cfe2
-  data.tar.gz: b79344d44bf6fd32cd9443c79405dcab052867a746e23fecc0b1496bd96b36fc6c4dd09d2a8c3d33389d7e5b5daaf0ca4a0552cce8948c59ac590da458c6150d
+  metadata.gz: 42c3be051514680426bed25b66ca09b8c420f1423a778db10512c37663c42128136acf6f4e72847bed68d4111e8563ef5eb1cd427883bf2802e072853f1900b0
+  data.tar.gz: d26d1c98f90f7bbd15474ac45f200a483983daadfae40016e5f9d2f0ca44b20a652e480f8bbfc13d939f904140358a1acc741fcbbb09306f35b51a298f3a9cf1

data/lib/twisty_puzzles/abstract_move.rb

@@ -143,6 +143,10 @@ module TwistyPuzzles
       prepend_slice_move(other, cube_size)
     end
 
+    def prepend_slice_move(_other, _cube_size)
+      raise NotImplementedError, "#{self.class}#prepend_slice_move is not implemented"
+    end
+
     private
 
     def move_count_internal(metric, slice_factor, direction_factor)

data/lib/twisty_puzzles/abstract_move_parser.rb

@@ -31,14 +31,14 @@ module TwistyPuzzles
     def parse_move(move_string)
       match = move_string.match(regexp)
       if !match || !match.pre_match.empty? || !match.post_match.empty?
-        raise ArgumentError("Invalid move #{move_string}.")
+        raise ArgumentError, "Invalid move #{move_string}."
       end
 
       parsed_parts = parse_named_captures(match)
       move_type_creators.each do |parser|
         return parser.create(parsed_parts) if parser.applies_to?(parsed_parts)
       end
-      raise "No move type creator applies to #{parsed_parts}"
+      raise ArgumentError, "No move type creator applies to #{parsed_parts}"
     end
   end
 end

data/lib/twisty_puzzles/algorithm_transformation.rb

@@ -4,28 +4,36 @@ require 'twisty_puzzles/cube_direction'
 require 'twisty_puzzles/rotation'
 
 module TwistyPuzzles
-  AlgorithmTransformation =
-    Struct.new(:rotation, :mirror, :mirror_normal_face) do
-      def transformed(algorithm)
-        algorithm = algorithm.mirror(mirror_normal_face) if mirror
-        algorithm.rotate_by(rotation)
-      end
+  # A transformation consisting of mirroring and rotating that can be applied to an algorithm.
+  class AlgorithmTransformation
+    def initialize(rotation, mirror, mirror_normal_face)
+      @rotation = rotation
+      @mirror = mirror
+      @mirror_normal_face = mirror_normal_face
+    end
 
-      def identity?
-        rotation.identity? && !mirror
-      end
+    attr_reader :rotation, :mirror, :mirror_normal_face
 
-      # Returns algorithm transformations that mirror an algorithm and rotate it around a face.
-      def self.around_face(face)
-        around_face_rotations = CubeDirection::ALL_DIRECTIONS.map { |d| Rotation.new(face, d) }
-        mirror_normal_face = face.neighbors.first
-        around_face_rotations.product([true, false]).map do |r, m|
-          AlgorithmTransformation.new(r, m, mirror_normal_face)
-        end
-      end
+    def transformed(algorithm)
+      algorithm = algorithm.mirror(mirror_normal_face) if mirror
+      algorithm.rotate_by(rotation)
+    end
 
-      def self.around_face_without_identity(face)
-        around_face(face).reject(&:identity?)
+    def identity?
+      rotation.identity? && !mirror
+    end
+
+    # Returns algorithm transformations that mirror an algorithm and rotate it around a face.
+    def self.around_face(face)
+      around_face_rotations = CubeDirection::ALL_DIRECTIONS.map { |d| Rotation.new(face, d) }
+      mirror_normal_face = face.neighbors.first
+      around_face_rotations.product([true, false]).map do |r, m|
+        AlgorithmTransformation.new(r, m, mirror_normal_face)
       end
     end
+
+    def self.around_face_without_identity(face)
+      around_face(face).reject(&:identity?)
+    end
+  end
 end

data/lib/twisty_puzzles/axis_face_and_direction_move.rb

@@ -55,7 +55,9 @@ module TwistyPuzzles
         self
       else
         rotation_neighbors = rotation.axis_face.neighbors
-        face_index = rotation_neighbors.index(@axis_face) || raise
+        face_index = rotation_neighbors.index(@axis_face)
+        raise unless face_index
+
         new_axis_face =
           rotation_neighbors[(face_index + rotation.direction.value) % rotation_neighbors.length]
         fields = replace_once(identifying_fields, @axis_face, new_axis_face)

data/lib/twisty_puzzles/color_scheme.rb

@@ -37,7 +37,7 @@ module TwistyPuzzles
     end
 
     def part_for_colors(part_type, colors)
-      raise ArgumentError unless part_type.is_a?(Class)
+      raise ArgumentError unless part_type.is_a?(Class) && (part_type < Part)
 
       part_type.for_face_symbols(colors.map { |c| face_symbol(c) })
     end
@@ -95,10 +95,12 @@ module TwistyPuzzles
 
     def chirality_corner_source_and_unknown_index(obvious_turned_face_symbols_to_colors)
       corner_matcher =
-        CornerMatcher.new(CHIRALITY_FACE_SYMBOLS.map do |s|
-                            # This will return nil for exactly one face that we don't know yet.
-                            @colors_to_face_symbols[obvious_turned_face_symbols_to_colors[s]]
-                          end)
+        CornerMatcher.new(
+          CHIRALITY_FACE_SYMBOLS.map do |s|
+            # This will return nil for exactly one face that we don't know yet.
+            @colors_to_face_symbols[obvious_turned_face_symbols_to_colors[s]]
+          end
+        )
 
       # There should be exactly one corner that gets mapped to the chirality corner.
       chirality_corner_source =

data/lib/twisty_puzzles/commutator.rb

@@ -10,6 +10,10 @@ module TwistyPuzzles
     def cancellations(other, cube_size, metric = :htm)
       algorithm.cancellations(other.algorithm, cube_size, metric)
     end
+
+    def algorithm
+      raise NotImplementedError
+    end
   end
 
   # Algorithm that is used like a commutator but actually isn't one.

data/lib/twisty_puzzles/coordinate.rb

@@ -6,7 +6,6 @@ require 'twisty_puzzles/native'
 
 module TwistyPuzzles
   # Coordinate of a sticker on the cube.
-  # rubocop:disable  Metrics/ClassLength
   class Coordinate
     def self.highest_coordinate(cube_size)
       cube_size - 1
@@ -122,23 +121,6 @@ module TwistyPuzzles
       from_indices(face, cube_size, m, m)
     end
 
-    def self.face(face, cube_size)
-      neighbor_a, neighbor_b = face.neighbors[0..1]
-      coordinate_range(cube_size).collect_concat do |x|
-        coordinate_range(cube_size).map do |y|
-          from_face_distances(face, cube_size, neighbor_a => x, neighbor_b => y)
-        end
-      end
-    end
-
-    def self.layer(face, cube_size)
-      face.neighbors.zip(face.neighbors.rotate(1)).collect_concat do |neighbor, next_neighbor|
-        coordinate_range(cube_size).map do |i|
-          from_face_distances(neighbor, cube_size, face => 0, next_neighbor => i)
-        end
-      end + self.face(face, cube_size)
-    end
-
     def self.edges_outside(face, cube_size)
       face.neighbors.zip(face.neighbors.rotate(1)).collect_concat do |neighbor, next_neighbor|
         1.upto(cube_size - 2).map do |i|
@@ -279,7 +261,6 @@ module TwistyPuzzles
       rots
     end
   end
-  # rubocop:enable Metrics/ClassLength
 
   # Coordinate of a sticker on the Skewb.
   class SkewbCoordinate

data/lib/twisty_puzzles/cube.rb

@@ -64,10 +64,18 @@ module TwistyPuzzles
       true
     end
 
+    def num_incarnations(_cube_size)
+      1
+    end
+
     def base_index_on_face(cube_size, incarnation_index)
       base_index_on_other_face(solved_face, cube_size, incarnation_index)
     end
 
+    def base_index_on_other_face(face, cube_size, incarnation_index)
+      raise NotImplementedError
+    end
+
     def self.for_face_symbols_internal(face_symbols)
       raise unless face_symbols.length == self::FACES
 
@@ -82,6 +90,10 @@ module TwistyPuzzles
       self::ELEMENTS[index]
     end
 
+    def self.valid?(_face_symbols)
+      false
+    end
+
     def <=>(other)
       @piece_index <=> other.piece_index
     end
@@ -110,7 +122,7 @@ module TwistyPuzzles
     # Rotate a piece such that the given face symbol is the first face symbol.
     def rotate_face_symbol_up(face_symbol)
       index = @face_symbols.index(face_symbol)
-      raise "Part #{self} doesn't have face symbol #{c}." unless index
+      raise "Part #{self} doesn't have face symbol #{face_symbol}." unless index
 
       rotate_by(index)
     end
@@ -133,10 +145,6 @@ module TwistyPuzzles
       (0...@face_symbols.length).map { |i| rotate_by(i) }
     end
 
-    def self.create_for_face_symbols(face_symbols)
-      new(face_symbols)
-    end
-
     def self.parse(piece_description)
       face_symbols =
         piece_description.upcase.strip.split('').map do |e|
@@ -338,10 +346,6 @@ module TwistyPuzzles
       find_only(self::ELEMENTS) { |e| e.corresponding_part == corresponding_part }
     end
 
-    def self.create_for_face_symbols(face_symbols)
-      new(self::CORRESPONDING_PART_CLASS.create_for_face_symbols(face_symbols))
-    end
-
     def face_symbol
       @face_symbols[0]
     end
@@ -514,7 +518,7 @@ module TwistyPuzzles
     end
 
     def num_incarnations(cube_size)
-      [cube_size / 2 - 1, 0].max
+      cube_size > 3 ? cube_size / 2 - 1 : 0
     end
 
     # One index of such a piece on a on a NxN face.
@@ -530,14 +534,6 @@ module TwistyPuzzles
   class Corner < Part
     FACES = 3
 
-    def self.create_for_face_symbols(face_symbols)
-      piece_candidates =
-        face_symbols[1..].permutation.map do |cs|
-          new([face_symbols[0]] + cs)
-        end
-      find_only(piece_candidates, &:valid?)
-    end
-
     def self.for_face_symbols(face_symbols)
       unless face_symbols.length == FACES
         raise "Invalid number of face_symbols to create a corner: #{face_symbols.inspect}"
@@ -621,7 +617,7 @@ module TwistyPuzzles
     ELEMENTS = generate_parts
 
     def num_incarnations(cube_size)
-      [cube_size / 2 - 1, 0].max
+      cube_size > 3 ? cube_size / 2 - 1 : 0
     end
 
     # One index of such a piece on a on a NxN face.
@@ -644,10 +640,10 @@ module TwistyPuzzles
     end
 
     def num_incarnations(cube_size)
-      if cube_size.even?
+      if cube_size.even? || cube_size <= 3
         0
       else
-        [cube_size / 2 - 1, 0].max
+        cube_size / 2 - 1
       end
     end
 

data/lib/twisty_puzzles/cube_state.rb

@@ -75,6 +75,16 @@ module TwistyPuzzles
 
     alias == eql?
 
+    def rotation_algorithms
+      Rotation::ALL_ROTATIONS.combination(2).reject do |r0, r1|
+        r0.inverse == r1
+      end.map { |rs| Algorithm.new(rs) } # rubocop:disable Style/MultilineBlockChain
+    end
+
+    def equal_modulo_rotations?(other)
+      rotation_algorithms.any? { |r| r.apply_temporarily_to(self) { |state| state == other } }
+    end
+
     def hash
       @hash ||= [self.class, @native].hash
     end

data/lib/twisty_puzzles/rotation.rb

@@ -35,10 +35,12 @@ module TwistyPuzzles
 
       direction = translated_direction(skewb_direction)
 
-      Algorithm.new([
-                      Rotation.new(corner.faces[skewb_direction.value], direction),
-                      Rotation.new(corner.faces[0], direction)
-                    ])
+      Algorithm.new(
+        [
+          Rotation.new(corner.faces[skewb_direction.value], direction),
+          Rotation.new(corner.faces[0], direction)
+        ]
+      )
     end
 
     def to_s

data/lib/twisty_puzzles/skewb_state.rb

@@ -26,10 +26,12 @@ module TwistyPuzzles
               next unless c2.face_symbols.include?(c1_rot.face_symbols.first)
 
               c2_rot = c2.rotate_face_symbol_up(c1_rot.face_symbols.first)
-              check_parts.push([
-                                 SkewbCoordinate.for_corner(c1_rot),
-                                 SkewbCoordinate.for_corner(c2_rot)
-                               ])
+              check_parts.push(
+                [
+                  SkewbCoordinate.for_corner(c1_rot),
+                  SkewbCoordinate.for_corner(c2_rot)
+                ]
+              )
             end
             matching_corners.push(check_parts)
           end

data/lib/twisty_puzzles/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module TwistyPuzzles
-  VERSION = '0.0.9'
+  VERSION = '0.0.10'
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: twisty_puzzles
 version: !ruby/object:Gem::Version
-  version: 0.0.9
+  version: 0.0.10
 platform: ruby
 authors:
 - Bernhard F. Brodowsky