[BUG]

[siavashk]

[BUG]

[gattia]

Thanks for the feedback! It looks like you might be using an old version (maybe installed via pypi? - sorry, this probably hasn't been updated in a while). I say that because if you look at the code on the lines you are throwing errors the code doesn't match:

pycpd/pycpd/deformable_registration.py

Lines 68 to 100 in e5ca02d

	np.linalg.solve((self.alpha * self.sigma2 * self.inv_S + np.matmul(self.Q.T, dPQ)),
	(np.matmul(self.Q.T, F))))))
	QtW = np.matmul(self.Q.T, self.W)
	self.E = self.E + self.alpha / 2 * np.trace(np.matmul(QtW.T, np.matmul(self.S, QtW)))
	def transform_point_cloud(self, Y=None):
	"""
	Update a point cloud using the new estimate of the deformable transformation.
	Attributes
	----------
	Y: numpy array, optional
	Array of points to transform - use to predict on new set of points.
	Best for predicting on new points not used to run initial registration.
	If None, self.Y used.
	Returns
	-------
	If Y is None, returns None.
	Otherwise, returns the transformed Y.
	"""
	if Y is not None:
	G = gaussian_kernel(X=Y, beta=self.beta, Y=self.Y)
	return Y + np.dot(G, self.W)
	else:
	if self.low_rank is False:
	self.TY = self.Y + np.dot(self.G, self.W)
	elif self.low_rank is True:
	self.TY = self.Y + np.matmul(self.Q, np.matmul(self.S, np.matmul(self.Q.T, self.W)))
	return

Can you trying installing from source? Or if you are, can you maybe update?

Thanks.

[a2mattreed]

Wow! Thanks for the amazing fast response. Indeed I’d installed from pypi using pip, but also had the source on hand, so swapped that in and it’s all good. I really appreciate your work on this library! —Matt

…

On Feb 26, 2023, at 1:33 PM, Anthony ***@***.***> wrote: Thanks for the feedback! It looks like you might be using an old version (maybe installed via pypi? - sorry, this probably hasn't been updated in a while). I say that because if you look at the code on the lines you are throwing errors the code doesn't match: https://github.com/siavashk/pycpd/blob/e5ca02d2501fb4b633c1664de939c336ffa2349e/pycpd/deformable_registration.py#L68-L100 Can you trying installing from source? Or if you are, can you maybe update? Thanks. — Reply to this email directly, view it on GitHub <#81 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AMBONUBVOZ7IMZ4E3H4AXQTWZOOWDANCNFSM6AAAAAAVITLS3Y>. You are receiving this because you authored the thread.

[gattia]

Glad it worked as expected with the source!

We'll work at updating the pypi version so others dont have this issue. Thanks!