twisty_puzzles 0.0.8 → 0.0.12

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: be6ba7974d92dd52a2b5b6307c38aae5695147003ed5f41c4fc36862ada74fb9
-  data.tar.gz: 8a6e4e82d8a76432c4a4e526fa664886aa2da41e4018ff260894cb4f3f96daec
+  metadata.gz: 78089ff494b1a687a567d483eb155e9891c69a991e8f4b045bf024e7365c58df
+  data.tar.gz: 820b61382a22d1c283f552ee637507dd9a511efa1d7f66656decd64f67304c8b
 SHA512:
-  metadata.gz: 478cc39f6e3e15184cac71f77680165735638a20f4fa431a311f544e0c0a44ea344d12d043fa318e6bbfd25dc76af7c2baef90ae286612c4d8cf18921c371520
-  data.tar.gz: d2b1dbefc310a59df6fe927951d8e1fdec5d553f6a4ce4d54c6caec51f0d7a69e688fedac652f32153ebbeb71459fd2f2e042d65f1febd45169c8f8a5f6a8cb6
+  metadata.gz: 063d2c8607b7d7708d0dea3fa78eebe3a883135cf2e2a46e5db7d5e0b80f6d21dd084b7a554718b54d45866eeecddf784857eb247a4a1de4ac924b8293aedd50
+  data.tar.gz: 9dff522e48073f756d4aa0e2377ff754b8fbefe0caaf8fb830383e47d14be7bb354e55c1ef2355fca1259dc255002e732f5089bd9269727973e2979de6a9d6e2

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 =
@@ -141,7 +143,7 @@ module TwistyPuzzles
                 "Color #{c} cannot be part of the color scheme because it is a reserved color."
         end
       end
-      raise ArgumentError unless face_symbols_to_colors.values.all? { |c| c.is_a?(Symbol) }
+      raise ArgumentError unless face_symbols_to_colors.values.all?(Symbol)
     end
 
     def add_missing_mappings(turned_face_symbols_to_colors, chirality_corner_source, unknown_index)

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,6 +6,7 @@ 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
@@ -27,7 +28,7 @@ module TwistyPuzzles
 
     # Middle coordinate for uneven numbers, the one before for even numbers
     def self.middle_or_before(cube_size)
-      cube_size - cube_size / 2 - 1
+      cube_size - (cube_size / 2) - 1
     end
 
     # Middle coordinate for uneven numbers, the one after for even numbers
@@ -37,7 +38,7 @@ module TwistyPuzzles
 
     # The last coordinate that is strictly before the middle
     def self.last_before_middle(cube_size)
-      cube_size / 2 - 1
+      (cube_size / 2) - 1
     end
 
     def self.canonicalize(index, cube_size)
@@ -121,6 +122,23 @@ 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|
@@ -261,6 +279,7 @@ module TwistyPuzzles
       rots
     end
   end
+  # rubocop:enable Metrics/ClassLength
 
   # Coordinate of a sticker on the Skewb.
   class SkewbCoordinate
@@ -291,7 +310,7 @@ module TwistyPuzzles
 
     def self.for_corner(corner)
       native = Native::SkewbCoordinate.for_corner(corner.face_symbols)
-      new(Face.for_face_symbol(corner.face_symbols.first), 1 + corner.piece_index % 4, native)
+      new(Face.for_face_symbol(corner.face_symbols.first), 1 + (corner.piece_index % 4), native)
     end
 
     def hash

data/lib/twisty_puzzles/cube.rb

@@ -34,6 +34,14 @@ module TwistyPuzzles
 
     attr_reader :piece_index, :face_symbols
 
+    def self.min_parseable_face_symbols
+      self::FACES
+    end
+
+    def self.max_parseable_face_symbols
+      self::FACES
+    end
+
     def self.generate_parts
       valid_face_symbol_combinations =
         FACE_SYMBOLS.permutation(self::FACES).select do |p|
@@ -64,10 +72,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 +98,10 @@ module TwistyPuzzles
       self::ELEMENTS[index]
     end
 
+    def self.valid?(_face_symbols)
+      false
+    end
+
     def <=>(other)
       @piece_index <=> other.piece_index
     end
@@ -110,7 +130,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,13 +153,9 @@ 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|
+        piece_description.upcase.strip.chars.map do |e|
           FACE_SYMBOLS[FACE_NAMES.index(e)]
         end
       for_face_symbols(face_symbols)
@@ -319,6 +335,14 @@ module TwistyPuzzles
   class MoveableCenter < Part
     FACES = 1
 
+    def self.min_parseable_face_symbols
+      self::CORRESPONDING_PART_CLASS::FACES
+    end
+
+    def self.max_parseable_face_symbols
+      self::CORRESPONDING_PART_CLASS::FACES
+    end
+
     def self.min_cube_size
       4
     end
@@ -338,10 +362,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
@@ -463,11 +483,15 @@ module TwistyPuzzles
       false
     end
 
+    def self.max_parseable_face_symbols
+      FACES + 1
+    end
+
     def self.for_face_symbols(face_symbols)
       # One additional face symbol is usually mentioned for wings.
       raise unless face_symbols.length == FACES || face_symbols.length == FACES + 1
 
-      if face_symbols.length == 3
+      if face_symbols.length == FACES + 1
         for_corner_face_symbols(face_symbols)
       else
         for_face_symbols_internal(face_symbols)
@@ -514,7 +538,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 +554,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 +637,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 +660,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_move.rb

@@ -68,8 +68,8 @@ module TwistyPuzzles
 
     def equivalent_slice_move?(other, cube_size)
       cube_size == 3 && other.slice_index == 1 &&
-        (@axis_face == other.axis_face && @direction == other.direction ||
-         @axis_face == other.axis_face.opposite && @direction == other.direction.inverse)
+        ((@axis_face == other.axis_face && @direction == other.direction) ||
+         (@axis_face == other.axis_face.opposite && @direction == other.direction.inverse))
     end
   end
 

data/lib/twisty_puzzles/cube_print_helper.rb

@@ -72,9 +72,9 @@ module TwistyPuzzles
     def skewb_ascii_art_line(first_color, middle_color, last_color, num_first_color)
       raise if num_first_color > SKEWB_FACE_SIZE / 2
 
-      first_color * num_first_color +
-        middle_color * (SKEWB_FACE_SIZE - 2 * num_first_color) +
-        last_color * num_first_color
+      (first_color * num_first_color) +
+        (middle_color * (SKEWB_FACE_SIZE - (2 * num_first_color))) +
+        (last_color * num_first_color)
     end
 
     def skewb_ascii_art(center_color, corner_colors)
@@ -116,7 +116,7 @@ module TwistyPuzzles
       front_face = face_lines(cube_state, :F, 1, 3) { |c| color_character(c, color_mode) }
       right_face = face_lines(cube_state, :R, 1, 3) { |c| color_character(c, color_mode) }
       pll_line = front_face.first + right_face.first
-      (top_face + [pll_line] * 3).join("\n")
+      (top_face + ([pll_line] * 3)).join("\n")
     end
 
     def cube_string(cube_state, color_mode)
@@ -150,7 +150,7 @@ module TwistyPuzzles
     end
 
     def pad_lines(lines, padding)
-      lines.map { |line| empty_name * padding + line }
+      lines.map { |line| (empty_name * padding) + line }
     end
 
     def zip_concat_lines(*args)

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/move_type_creator.rb

@@ -6,7 +6,7 @@ module TwistyPuzzles
   class MoveTypeCreator
     def initialize(capture_keys, move_class)
       raise TypeError unless move_class.is_a?(Class)
-      raise TypeError unless capture_keys.all? { |k| k.is_a?(Symbol) }
+      raise TypeError unless capture_keys.all?(Symbol)
 
       @capture_keys = capture_keys.freeze
       @move_class = move_class

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.8'
+  VERSION = '0.0.12'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: twisty_puzzles
 version: !ruby/object:Gem::Version
-  version: 0.0.8
+  version: 0.0.12
 platform: ruby
 authors:
 - Bernhard F. Brodowsky
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2020-04-21 00:00:00.000000000 Z
+date: 2021-11-01 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: colorize