FuzzyClustering icon indicating copy to clipboard operation
FuzzyClustering copied to clipboard

Using smaller root than power?

Open Make42 opened this issue 5 years ago • 1 comments

In fcm_get_u you have the line inv_distances = np.reciprocal(nonzero_distances) ** (1 / (m - 1)). Shouldn't m-1 be replaced with m? In fcm you write um = u ** m afterall.

Make42 avatar Mar 02 '21 10:03 Make42

I think there was some issue with literature where m was used differently depending on author. Not sure though if this was the exact issue.

The derivation went something like this. The -1 in the computation of u comes from taking the derivative with respect to u. When computing v, there is no derivative with respect to u, so no -1 there.

But maybe there is a mistake in there somewhere, so if you find any, feel free to point them out.

derivation

99991 avatar Mar 02 '21 14:03 99991