Skip to content

Commit f81ac60

Browse files
author
John Halloran
committed
style: remove references to old variable names
1 parent 640dacf commit f81ac60

File tree

1 file changed

+11
-11
lines changed

1 file changed

+11
-11
lines changed

src/diffpy/snmf/snmf_class.py

Lines changed: 11 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -636,19 +636,19 @@ def regularize_function(self, stretch=None):
636636
if stretch is None:
637637
stretch = self.stretch
638638

639-
K = self.n_components
640-
M = self.n_signals
641-
N = self.signal_length
642-
643639
stretched_components, d_stretch_comps, dd_stretch_comps = self.apply_interpolation_matrix(stretch=stretch)
644-
intermediate = stretched_components.flatten(order="F").reshape((N * M, K), order="F")
645-
residuals = intermediate.sum(axis=1).reshape((N, M), order="F") - self.source_matrix
640+
intermediate = stretched_components.flatten(order="F").reshape(
641+
(self.signal_length * self.n_signals, self.n_components), order="F"
642+
)
643+
residuals = (
644+
intermediate.sum(axis=1).reshape((self.signal_length, self.n_signals), order="F") - self.source_matrix
645+
)
646646

647647
fun = self.get_objective_function(residuals, stretch)
648648

649-
tiled_res = np.tile(residuals, (1, K))
649+
tiled_res = np.tile(residuals, (1, self.n_components))
650650
grad_flat = np.sum(d_stretch_comps * tiled_res, axis=0)
651-
gra = grad_flat.reshape((M, K), order="F").T
651+
gra = grad_flat.reshape((self.n_signals, self.n_components), order="F").T
652652
gra += self.rho * stretch @ (self._spline_smooth_operator.T @ self._spline_smooth_operator)
653653

654654
# Hessian would go here
@@ -657,10 +657,10 @@ def regularize_function(self, stretch=None):
657657

658658
def update_stretch(self):
659659
"""
660-
Updates matrix A using constrained optimization (equivalent to fmincon in MATLAB).
660+
Updates stretching matrix using constrained optimization (equivalent to fmincon in MATLAB).
661661
"""
662662

663-
# Flatten A for compatibility with the optimizer (since SciPy expects 1D input)
663+
# Flatten stretch for compatibility with the optimizer (since SciPy expects 1D input)
664664
stretch_flat_initial = self.stretch.flatten()
665665

666666
# Define the optimization function
@@ -682,7 +682,7 @@ def objective(stretch_vec):
682682
bounds=bounds,
683683
)
684684

685-
# Update A with the optimized values
685+
# Update stretch with the optimized values
686686
self.stretch = result.x.reshape(self.stretch.shape)
687687

688688

0 commit comments

Comments
 (0)