Support dense matrices or convert to sparse instead of raising TypeError
macks22 opened this issue · 1 comments
macks22 commented
When I try to fit als.FMClassifier with a dense numpy matrix, I get:
File "/home/mack/anaconda2/lib/python2.7/site-packages/fastFM/als.py", line 175, in fit
order="F")
File "/home/mack/anaconda2/lib/python2.7/site-packages/fastFM/validation.py", line 29, in wrapper
raise TypeError('A dense matrix was passed in, but sparse'
TypeError: A dense matrix was passed in, but sparsedata is required.
Any reason not to support dense matrices? I'm wondering if it's simply because the type of data an FM is well-suited for cannot generally be represented by a dense matrix. Seems like if the user can fit it into memory, then why not support it? If sparse matrices are absolutely required, is it reasonable to simply convert to a dense matrix instead of raising the TypeError
?
macks22 commented
Sorry, wrong repo!