Predict ratingmatrix about recommenderlab HOT 4 CLOSED

This makes sense. I have changed it on GitHub. Please install the package from there for now. It will be part of the next scheduled release.

from recommenderlab.

I'm currently looking at the code in predict.R and trying to figure out what line 23 and 25 are for. I think that removing these lines will solve the problem: the ratings in "rm" are already normalized and consist of (I think?) the model's predictions. No need to do any further evaluations using the newdata variable, right? The matrix rm is already the desired output as far as I can judge

from recommenderlab.

That is correct. I am currently thinking about what the right thing to do is here. Maybe I should not replace the predictions with the known values.

from recommenderlab.

Thank you for looking into this. I think removing the lines I mentioned should do the trick, and as you said: replacing the predictions with known values does not make sense, as they are predictions and not ground truths :). It would be much appreciated if this can be changed in the code soon, as I am highly dependent of the software for my current research.

from recommenderlab.