undefined method `reduce' about classifier-reborn HOT 10 CLOSED

@jjk-jacky That's my commit, so I'll take a look and see if I can reproduce and fix the issue.

from classifier-reborn.

Thanks @Ch4s3!

from classifier-reborn.

I can't reproduce the error with gsl 1.16 on OSX yet, but a simple fix would be to add a check for GSL here and using the old code for GSL vectors and the new code for vector arrays. I'm not sure I like that though, so I'm going to keep testing and think it over.

from classifier-reborn.

I'll let you take the lead on this one, @Ch4s3. Thanks!

from classifier-reborn.

I don't know much about all this, but looking into it a bit, I found this issue which seems to indicate, as the error I reported, that the GSL::Vector object doesn't have a method reduce -- Though I'm not sure why you can't reproduce it then, if using GSL?

Anyways, I'd guess then that in both linked commits it might indeed be needed to checked whether it is a GSL::Vector used, and if so keep the use of sum.

from classifier-reborn.

This is interesting, it seems that GSL::Vector supports .map and .collect 555d4a3 was about removing that monkey patch. I'm using a check for GSL and .collect to get the sum. the pull is here

from classifier-reborn.

In https://github.com/FredrikWendt/jekyll-issue I have this problem in a Docker environment (so it would be easy to reproduce).

from classifier-reborn.

@FredrikWendt, are you using rb-gsl?

from classifier-reborn.

Yes I am. And the above issue only appears when run with classifier-reborn version 2.0.2. In the Docker environment 2.0.3 is installed now. I won't spend more time trying to fix this - I included a patch which I ran on /var/lib/gems/1.9.1/gems/classifier-reborn-2.0.2/lib/classifier-reborn/lsi/content_node.rb before invoking jekyll.

from classifier-reborn.

2.0.3 should fix this.

from classifier-reborn.