VMF scaling denominator was inf about spherecluster HOT 8 OPEN

It would be great to know a bit about the data you're working with. That exception is raised when the dimension is too large.

from spherecluster.

I experience the same problem when the dimension is relatively low. For example, the test test_integration_dense in test_von_mises_fisher_mixture.py raises this exception most of the time if I let n_features = 10 (and fixes params_in to {"posterior_type": "soft"}).

from spherecluster.

see my comment here #17
EDIT
It is actually straight forward to work in log scale using the exponentially-scaled modified Bessel functions:

from numpy import i0e  # exp-scaled modified Bessel function of first kind order 0, I_0                                                                                                                               
from scipy.special import ive  # exp-scaled modified Bessel function of first kind, I_v   

from spherecluster.

Thanks @itamar-dw would you like to add a PR for this change?

from spherecluster.

sure.
I tried to create a new branch and push it, but I got:

remote: Permission to jasonlaska/spherecluster.git denied to itamar-dw

from spherecluster.

@itamar-dw I saw one branch come up but it was derived off of master. Could you try branching off develop (pull the latest develop first or rebase), then tag me on the PR? I will merge it in after testing and reviewing.

You should be able to submit a PR for this project via the github UI by forking to your own account first https://git-scm.com/book/en/v2/GitHub-Contributing-to-a-Project

Let me know if you need help.

from spherecluster.

Alternatively, I'm happy to cherry-pick from a gist, if that's easier.

from spherecluster.

Yes I created a PR then I realized I have to create a branch first. I'll do it as you suggested, that's an opportunity to learn how to do it :)

from spherecluster.