Rename confusing variables

Author: RotemKalisch

ramanujantools/asymptotics/reducer.py

@@ -230,15 +230,17 @@ def shear(self) -> None:
             self._is_reduced = True
             return
 
-        if not g.is_integer:
-            g, b = g.as_numer_denom()
-            self.M, self.S_total = self.M.ramify(b), self.S_total.ramify(b)
-            self.p *= b
-            self.precision *= b
+        shift, ramification = g.as_numer_denom()
+        self.M, self.S_total = (
+            self.M.ramify(ramification),
+            self.S_total.ramify(ramification),
+        )
+        self.p *= ramification
+        self.precision *= ramification
 
-        true_valid_precision, max_shift = self._check_shear_truncation(g)
+        true_valid_precision, max_shift = self._check_shear_truncation(shift)
 
-        logger.debug(f"SHEAR: Computed slope {g=}. Max shift: {max_shift=} terms.")
+        logger.debug(f"SHEAR: Computed shift {shift=}. Max shift: {max_shift=} terms.")
         if max_shift > 0:
             padded_coeffs = (
                 self.S_total.coeffs + [Matrix.zeros(self.dim, self.dim)] * max_shift
@@ -252,12 +254,12 @@ def shear(self) -> None:
             f"Remaining buffer: {self.S_total.precision - max_shift}"
         )
 
-        S_sym = Matrix.diag(*[t ** (i * g) for i in range(self.dim)])
+        S_sym = Matrix.diag(*[t ** (i * shift) for i in range(self.dim)])
         S_series = SeriesMatrix.from_matrix(S_sym, n, self.p, self.S_total.precision)
 
         self.S_total = self.S_total * S_series
 
-        self.M, h = self.M.shear_coboundary(g, true_valid_precision)
+        self.M, h = self.M.shear_coboundary(shift, true_valid_precision)
         self.precision = true_valid_precision
 
         if h != 0:

ramanujantools/asymptotics/series_matrix.py

@@ -244,22 +244,25 @@ def _shear_row_corrections(self, g: int) -> list[list[sp.Expr]]:
         return row_corrections
 
     def shear_coboundary(
-        self, g: sp.Rational | int, target_precision: int | None = None
+        self, shift: sp.Integer | int, target_precision: int | None = None
     ) -> tuple[SeriesMatrix, int]:
         """
         Applies a shearing transformation S(t) to the series to expose sub-exponential
-        growth, where $S(t) = diag(1, t^g, t^{2g}, \\dots)$.
+        growth, where $S(t) = diag(1, t^a, t^{2a}, \\dots)$ and $a$ is the shift parameter.
 
         Args:
-            g: The shear slope.
+            shift: Represents a in the above description.
             target_precision: If provided, truncates the resulting series to this length,
                               saving heavy CAS simplification on discarded tail terms.
 
         Returns:
             A tuple containing the sheared SeriesMatrix and the integer `h` representing
             the overall degree shift (used to adjust the global factorial power).
         """
-        row_corrections = self._shear_row_corrections(g)
+        if not shift.is_integer:
+            raise ValueError(f"{shift=} must be an integer!")
+
+        row_corrections = self._shear_row_corrections(shift)
         power_dict = {}
 
         for m in range(self.precision):
@@ -269,13 +272,13 @@ def shear_coboundary(
                     if val_M == sp.S.Zero:
                         continue
 
-                    shift = int((j - i) * g)
+                    current = (j - i) * shift
                     for c in range(self.precision):
                         val_C = row_corrections[i][c]
                         if val_C == sp.S.Zero:
                             continue
 
-                        power = m + c + shift
+                        power = m + c + current
                         if power not in power_dict:
                             power_dict[power] = Matrix.zeros(*self.shape)
                         power_dict[power][i, j] += val_C * val_M